Skip to content

fix(regex): load PPR large dynamic grammar - #1317

Merged
fglock merged 2 commits into
masterfrom
fix/issue-1299
Sep 9, 2026
Merged

fix(regex): load PPR large dynamic grammar#1317
fglock merged 2 commits into
masterfrom
fix/issue-1299

Conversation

@fglock

@fglock fglock commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • Replace exponential named-subexpression recursion analysis with SCC-based marking.
  • Preserve static infinite-recursion diagnostics while carrying runtime-callout provenance into Joni.
  • Add a scalable dynamic named-grammar regression and a changelog entry.

Fixes #1299.

Validation

  • make
  • timeout 60 ./jperl -I/tmp/ppr-1299/lib -e 'require PPR; print "ok\\n"'
  • timeout 60 ./jperl --interpreter -I/tmp/ppr-1299/lib -e 'require PPR; print "ok\\n"'
  • make check-links

Generated with Codex

Replace repeated recursive call-graph walks with component-based recursion
marking, and carry dynamic-callout provenance into Joni's static safety
analysis.  This lets PPR load while retaining static infinite-recursion
diagnostics.

Fixes #1299.

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock
fglock marked this pull request as ready for review September 9, 2026 09:06
Match jperl's 16 MB stack in Gradle Test workers so direct Perl unit
execution reaches the 1000-call compatibility guard on Windows.

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock
fglock merged commit ecde858 into master Sep 9, 2026
2 checks passed
@fglock
fglock deleted the fix/issue-1299 branch September 9, 2026 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Joni regex compilation hangs loading PPR 0.001010 on both PerlOnJava backends

1 participant