search

9fans / plan9port

Plan 9 from User Space
https://9fans.github.io/plan9port/
Other
1.61k stars 319 forks source link

libregexp: allow up to 127 captured subexpressions in one regexp #609

Closed igorburago closed 1 year ago

igorburago commented 1 year ago

128 counting with the entire expression match ($0).

The most straightforward way to address point 2 from #565 (in preparation for #610). Please let me know if 128 is not a good candidate for this bound, or if dynamic allocation would be preferable instead (as discussed in the linked issue).

igorburago commented 1 year ago

@dancrossnyc, it looks like this issue was left behind by mistake: #610 requires this one to work (as stated in its description), yet #610 is merged and this one is not!

dancrossnyc commented 1 year ago

Huh, I thought I had. My bad!