Menu

#11 Patch to remove the 255 character limit on string literal size

open
nobody
None
5
2020-08-28
2014-10-04
No

Hi
The attached patch allows the reader to read strings longer than 255 characters, by making the buffer size dynamic. It is doubled whenever it is found to be too small. We found we needed this for working with database queries, documentation strings etc. Do you think this is the right approach?
Thanks!
Thomas Munro

1 Attachments

Discussion

  • Justus Winter

    Justus Winter - 2016-02-24

    I'd like to see this patch merged. My workaround was to make that limit configurable, but this is much nicer of course.

    A little nitpick, I'd avoid "p-sc->strbuff" in favor of "p - sc->strbuff". I've seen that the original code used the former version too, but that looks just wrong to my eyes.

     
  • Thomas Munro

    Thomas Munro - 2020-08-28

    +1 to adding spaces around the minus operator :-) Would be great to see this and other old patches in the queue eventually land in official upstream TinyScheme 1.43.

     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.