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

[stmt.do] Add grammar equivalence and reorder paragraphs #6495

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Eisenwave
Copy link
Contributor

Fixes #6492 for the most part.

image

This edit improves a few things:

  1. The two paragraphs are swapped, which creates symmetry with how they are ordered in [stmt.while]
  2. Similar to while and for, the equivalence to a goto construct is also provided.
    Note that the trailing semicolon issue is swept under the rug by defining the equivalence for while (expression); with a semicolon present
  3. \grammarterm is used in a few more places.

@jensmaurer jensmaurer added the cwg Issue must be reviewed by CWG. label Feb 28, 2024
@jensmaurer
Copy link
Member

This shouldn't go in without CWG seeing it.

@wg21bot wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cwg Issue must be reviewed by CWG. needs rebase The pull request needs a git rebase to resolve merge conflicts.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[stmt.do] Needs some work
3 participants