Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix error reporting for one_or_more rule #5

Closed
obenjiro opened this issue Jun 27, 2016 · 1 comment
Closed

Fix error reporting for one_or_more rule #5

obenjiro opened this issue Jun 27, 2016 · 1 comment
Labels
bug

Comments

@obenjiro
Copy link
Member

Right now if we parse wrong PEG grammar we got this

Unexpected "s" expected (EOF)
1: GRAMMAR test a -> "A"; should_fail -> "B" -
--------------------------^

We should get more informative error message, like this

Unexpected "-" expected (;)
1: GRAMMAR test a -> "A"; should_fail -> "B" -
---------------------------------------------^
@obenjiro obenjiro added the bug label Jun 27, 2016
@obenjiro obenjiro mentioned this issue Jun 27, 2016
Open
2 tasks
obenjiro pushed a commit that referenced this issue Jun 28, 2016
Allow extra space at the beginning of the GRAMMAR #5
d5545a5
@obenjiro
Copy link
Member Author

fixed in 26bf50b

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@obenjiro