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

Generate precompiler build error reports #18

Closed
teo-tsirpanis opened this issue Feb 24, 2021 · 1 comment
Closed

Generate precompiler build error reports #18

teo-tsirpanis opened this issue Feb 24, 2021 · 1 comment
Assignees
Milestone

Comments

@teo-tsirpanis
Copy link
Owner

teo-tsirpanis commented Feb 24, 2021

As shown in #9, GOLD Parser displayes LALR conflicts in a not so helpful way. While Farkle 6.0.0 has somehow improved the error messages for them, there is still lots of room for improvement.

We can use the new HTML generator to also create error reports for failed builds, displaying the full LALR state table instead of just the conflicts without any context.

This feature would require changes to the way LALR conflicts are represented but that's an acceptable break. It would also require some changes in the LALR builder and conflict resolver.

@teo-tsirpanis teo-tsirpanis added this to the Farkle 6.1 milestone Feb 24, 2021
@teo-tsirpanis teo-tsirpanis self-assigned this Feb 24, 2021
@teo-tsirpanis
Copy link
Owner Author

Fixed in Farkle 6.3.0 where conflict reports were introduced.

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

No branches or pull requests

1 participant