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

Recoverable Errors #88

Open
tohrnii opened this issue Nov 29, 2022 · 2 comments
Open

Recoverable Errors #88

tohrnii opened this issue Nov 29, 2022 · 2 comments
Labels
enhancement New feature or request on hold parser

Comments

@tohrnii
Copy link
Contributor

tohrnii commented Nov 29, 2022

We should add recoverable errors to the parser to recover all errors while parsing instead of terminating on encountering the first error.

@bobbinth
Copy link
Contributor

bobbinth commented Jul 2, 2023

@bitwalker - I think we already have this? If so, we should probably close this issue.

@bitwalker
Copy link
Contributor

We've relaxed the parser a bit, and removed some of the hard errors, but a few things are still hard errors that stop parsing. We would need to use the error recovery features in LALRPOP to make parsing complete before raising errors. We're pretty much set up for it now, just requires some modifications to the grammar file and AST to store error nodes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request on hold parser
Projects
None yet
Development

No branches or pull requests

3 participants