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.

Sign up for GitHub

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

Jump to bottom

improve error reporting for lookahead in macros #1089

Open
lsf37 opened this issue Apr 10, 2023 · 2 comments
Open

improve error reporting for lookahead in macros #1089

lsf37 opened this issue Apr 10, 2023 · 2 comments
Labels
enhancement Feature requests

Comments

@lsf37
Copy link
Member

lsf37 commented Apr 10, 2023

See #1088 -- instead of just a syntax error we should be getting an explanation.
@lsf37 lsf37 added the enhancement Feature requests label Apr 10, 2023
@lsf37 lsf37 mentioned this issue Apr 10, 2023
Closed
@lsf37
Copy link
Member Author

lsf37 commented Apr 10, 2023

Alternatively: allow lookahead in macros and at any nesting level syntactically, but add a check in the semantic checking phase that rules out expressions where lookahead is not used at the top level.

@michab66
Copy link

That alternative option would be great. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Feature requests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@michab66 @lsf37