-
Notifications
You must be signed in to change notification settings - Fork 116
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.
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
Performance problem with a grammar #213
Comments
The fix for
|
After seeing |
Could you spot which definition rule causes this performance problem? |
I'll try but so far compiling with profiling enabled and
|
The tests I did so far do not lead to one specific rule responsible for the long time processing, it seems that is somehow proportional to the grammar size. |
Also this grammar naively converted from
|
See also this comment ChrisHixon/chpeg#23 (comment) |
@mingodad I fixed it. Could you verify it with your benchmark test? Thanks! |
Thank you ! |
With the grammars attached I can use then with
Then on the play ground if I try only this grammar:
Then I get this error:
|
@mingodad, I can't reproduce the problem on playground. Also could you make a new separate issue for this problem since this has nothing to do with the performance problem and this issue has been already closed? |
With this grammar
chpeg
(dev playground https://meimporta.eu/chpeg/) detected infinite loop butcpp-peglib
doesn't and freeze the browser:The text was updated successfully, but these errors were encountered: