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

Cosider to migrate to pegjs 0.9 #4

Open
physikerwelt opened this issue Oct 17, 2015 · 1 comment
Open

Cosider to migrate to pegjs 0.9 #4

physikerwelt opened this issue Oct 17, 2015 · 1 comment

Comments

@physikerwelt
Copy link
Contributor

I tried to migrate to pegjs 0.9, but it seems that the following block

expr
  = ne_expr
  / /* */
  { return ast.LList.EMPTY; }

does not pass validation anymore.

45:3: Expected "!", "$", "&", "(", ".", character class, comment, end of line, identifier, literal or whitespace but "{" found.

Some of the changes from 0.8 to 0.9 seem to be interesting https://github.com/pegjs/pegjs/blob/master/CHANGELOG.md.

@physikerwelt
Copy link
Contributor Author

I had a look at that. One needs to replace empty tokens with "", change the name of the parse() function.
In addition, I removed the ..h version that were not used since they caused recursion problems.
The last thing I need to do is to update the error reporting. Afterwards I'll make a pull request

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

No branches or pull requests

1 participant