Skip to content

Fix typos in how_to_contradiction.v - #134

Open
amblafont wants to merge 1 commit into
rocq-prover:mainfrom
amblafont:patch-2
Open

Fix typos in how_to_contradiction.v#134
amblafont wants to merge 1 commit into
rocq-prover:mainfrom
amblafont:patch-2

Conversation

@amblafont

@amblafont amblafont commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

The tutorial may need some updates: it occasionally mentions features that are not used in this tutorial because they are not yet available, but I think now there are, since 9.2 is out

]]

Notation, to do [Control.hyp] and [$] at once is only available in Rocq 9.1 or above.
Notation to do [Control.hyp] and [$] at once is only available in Rocq 9.1 or above.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is this notation ?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SkySkimmer do you know, I don't remember

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably &
constr:(&x + 1) is roughly equivalent to let y := Control.hyp @x in constr:($y + 1) (and strictly equivalent to constr:(ltac2:(Control.refine (fun _ => Control.hyp @x))))

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

although that has existed for much longer than 9.1 so maybe not

@SkySkimmer SkySkimmer Aug 14, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking at the changelog for 9.1, probably rocq-prover/rocq#20656 $hyp:x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants