Skip to content
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

bc: yyparse() loop regression #619

Merged
merged 1 commit into from
May 24, 2024
Merged

Commits on May 24, 2024

  1. bc: yyparse() loop regression

    * A previous commit replaced a goto into yyparse() with a call to yyparse() but this introduced an infinite loop for a file with a syntax error
    * yy_err_recover() returns bool; returning 1 instead of calling yyparse() again allows yyparse() to itself return and break out of "yyloop"
    * The pattern of return(1) from yy_err_recover() was already established; this patch just makes it more consistent
    * test1: echo '1++' > A.bc && perl bc -y A.bc
    * test2: echo '~' > B.bc && perl  bc -y B.bc
    mknos committed May 24, 2024
    Configuration menu
    Copy the full SHA
    101d620 View commit details
    Browse the repository at this point in the history