Skip to content

[basic.scope.pdecl], [temp] Replace "expansion statement" with "expansion-statement"#8370

Merged
tkoeppe merged 1 commit into
cplusplus:mainfrom
eisenwave:cd26-404
Oct 31, 2025
Merged

[basic.scope.pdecl], [temp] Replace "expansion statement" with "expansion-statement"#8370
tkoeppe merged 1 commit into
cplusplus:mainfrom
eisenwave:cd26-404

Conversation

@eisenwave

@eisenwave eisenwave commented Oct 26, 2025

Copy link
Copy Markdown
Member

Fixes NB US 2-404 (C++26 CD).
Also fixes https://github.com/cplusplus/nbballot/issues/585.

This PR does a little bit more than the NB comment asked for. We have the bad language of

for-range-declaration of an expansion statement

and

compound-statement of an expansion statement

The NB comment only relates to the second case, but in general, if we talk about one grammatical thing being inside another, we should consistently use grammar font.

@eisenwave eisenwave added the ballot-comment Response to an NB or ISO comment on a ballot label Oct 26, 2025
…sion-statement"

Fixes NB US 2-404 (C++26 CD).

@jensmaurer jensmaurer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Those are all statements about grammar, so referring to the grammarterm expansion-statement is correct.

@tkoeppe tkoeppe merged commit fde9d1f into cplusplus:main Oct 31, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ballot-comment Response to an NB or ISO comment on a ballot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants