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

1729/1737 Fix grammar for "declare record" #1767

Open

michaelhkay wants to merge 2 commits into qt4cg:master from michaelhkay:1729-correction-to-declare-record-syntax

Contributor

michaelhkay commented Feb 5, 2025

The syntax should be "declare record", not "declare type record".
All the declarations using annotations should allow multiple annotations.
Added a note about refactoring the grammar to avoid unbounded lookahead.


          Fix grammar for "declare record"

df55dfc

ChristianGruen approved these changes

View reviewed changes

ChristianGruen added the Propose Merge without Discussion label

Contributor Author

michaelhkay commented Feb 5, 2025

Extended the PR to also fix #1737

michaelhkay changed the title ~~1729 Fix grammar for "declare record"~~ 1729/1737 Fix grammar for "declare record"


          Drop unused production rule AnnotationDecl

28079c5

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

Propose Merge without Discussion