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

Misparsed tactic leads to two commands being exeucted at once #895

Open
RalfJung opened this issue Sep 10, 2024 · 1 comment
Open

Misparsed tactic leads to two commands being exeucted at once #895

RalfJung opened this issue Sep 10, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@RalfJung
Copy link

To reproduce, consider:

Lemma L (P Q R: Prop) :
  (P -> Q -> R) -> P -> Q -> R.
Proof.
  intros Himpl HP HQ. (* Process proof until here *)
  speciaalize (Himpl HP HQ) as Goo.

Qed.

Now hit Alt-down. I would expect vscoq to try to process the next tactic, which has a typo in it and therefore doesn't parse. However, processing actually skips all the way to after the Qed and I get two errors:

image

That is quite confusing. Worse, the Coq proof state window only shows the second error, making the actual underlying cause of the problem invisible:
image

I can see it thanks to the error-lens addon, but if one does not have that installed, one has to notice the error in the "problems" panel.

The fact that according to the last screenshot, L got declared is concerning, since there can't even have been a complete proof.

This is with Coq 8.18.0.

@rtetley
Copy link
Collaborator

rtetley commented Sep 13, 2024

This is a duplicate of #877. I am busy working on a fix ! I will keep open because of the L is declared thing. This is probably related to the separation of interp and synterp phases.

@rtetley rtetley added the bug Something isn't working label Sep 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants