Nothing Special   »   [go: up one dir, main page]

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

Take advantage of JFlex's EOF handling for better error messages. #226

Merged
merged 1 commit into from
Jan 19, 2018

Conversation

guardbotmk3
Copy link

This is relative to my other PR, So only look at e6908cd

@andreasabel
Copy link
Member

This PR does not build since it still uses LineNumber.

@guardbotmk3
Copy link
Author

Sorry, that's now fixed.

@andreasabel andreasabel merged commit 7c9e859 into BNFC:master Jan 19, 2018
@andreasabel andreasabel added this to the 2.8.2 milestone Jan 19, 2018
@andreasabel andreasabel self-assigned this Jan 19, 2018
@andreasabel
Copy link
Member

Would you have a test case that demonstrates the better error message?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants