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

TLAPM parser does not accept implicit <*>/<+> proof step IDs with names #170

Closed
ahelwer opened this issue Oct 24, 2024 · 1 comment
Closed
Labels
bug An error, usually in the code. syntax parser Issues relating to TLAPM's syntax parser

Comments

@ahelwer
Copy link
Collaborator

ahelwer commented Oct 24, 2024

Related to (and possibly the cause of) #165. Whereas that issue was about referring to proof steps in the same level, this is about writing proof steps in a given level. The TLAPM parser does not accept names being added to steps with an implicit level. These two inputs will both fail:

---- MODULE Test ----
THEOREM TRUE
<*>abc 1
<*>def QED
====
---- MODULE Test ----
THEOREM TRUE
<1>abc 1
  <+>def 2
  <2>ghi QED
<1>jkl QED
====

Ref #159

@ahelwer ahelwer added bug An error, usually in the code. syntax parser Issues relating to TLAPM's syntax parser labels Oct 24, 2024
@ahelwer
Copy link
Collaborator Author

ahelwer commented Feb 5, 2025

Further experimentation with SANY's semantic checker in tlaplus/tlaplus#1133 shows this is expected behavior, and implicit steps should not have names; SANY gives this error:

<*> and <+> cannot be used for a named step.

These tests should be removed from the syntax corpus.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An error, usually in the code. syntax parser Issues relating to TLAPM's syntax parser
Development

No branches or pull requests

1 participant