News for hackers. Stuff that matters.

A comment on Show HN: Sed to C translator written in sed. Parent comment

Re:Show HN: Sed to C translator written in sed by segfaultbuserr on Sunday July 12, 2020 @01:44PM (#23813158)
Not the author, but I see that

> no pattern/hold space overflow checks, currently both limited to 8192 bytes as per the POSIX spec requirement. Going over that limit will most likely cause a segfault.

> The C code is very rough around the edges (by that I mean dirty and unsafe, for instance allocating everything on the stack without checking any overflow), I'm still working on it, but contributions (issues/comments/pull requests) are also welcomed :)

Thus, so far it's just for fun, not suitable for profit yet.

[ Parent | on HN ]
Re:Show HN: Sed to C translator written in sed by lhoursquentin on Sunday July 12, 2020 @02:56PM (#23813771)
Yep that's exactly right :) !

To clarify my statement POSIX requires at least 8192 bytes for the pattern and hold space, and I chose to allow at most 8192 bytes here, which indeed means that this will prevent working on really long lines or storing entire files in the hold space for instance.

[ Parent | on HN ]