New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix grammar compression regression #10

Open

JanJakes opened this issue Feb 10, 2025 · 0 comments · May be fixed by #11

Contributor

JanJakes commented Feb 10, 2025

There is a small regression in the grammar processing script:

The PHPized grammar now contains array indexes of the main rules, while previously they were not listed. It seems there are numeric gaps. It might be a regression caused when manually parsing the grammar. I suppose it's an easy fix.

This makes the compressed grammar a bit larger, as it includes array indexes that don't need to be indexed.

JanJakes linked a pull request

that will close this issue

Fix grammar compression #11

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment