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

Grammar failes to parse valid PlusCal syntax when eliding curly braces #92

Open
ahelwer opened this issue Jan 6, 2024 · 1 comment
Open
Assignees
Labels
bug Something isn't working pluscal Related to PlusCal parsing

Comments

@ahelwer
Copy link
Contributor

ahelwer commented Jan 6, 2024

Picked this up from a new submodule added to the tlaplus/examples repo:

https://github.com/tlaplus/azure-cosmos-tla/blob/4b8785a36ed16faa2cd1bd78b89e224d71711ee3/scenario1/swscop.tla#L34

Both pcal.trans and tla2sany.SANY accept this syntax. At first glance I think the grammar wants there to be curly braces, but they're elided here.

@susliko would you be able to take a look when you have time?

@ahelwer ahelwer added the bug Something isn't working label Jan 6, 2024
@ahelwer ahelwer changed the title Grammar failes to parse valid PlusCal syntax Grammar failes to parse valid PlusCal syntax when eliding curly braces Jan 6, 2024
@ahelwer ahelwer added the pluscal Related to PlusCal parsing label Apr 22, 2024
@ahelwer
Copy link
Contributor Author

ahelwer commented Apr 22, 2024

Tried simple fix but ran into a whole bunch of parser ambiguity issues. Work currently lives in pcal-no-delimiter branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working pluscal Related to PlusCal parsing
Projects
None yet
Development

No branches or pull requests

2 participants