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.

Sign up for GitHub

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

Jump to bottom

[stmt.expr] use grammarterm for expression #6469

Merged
merged 1 commit into from Dec 22, 2023
Merged

[stmt.expr] use grammarterm for expression #6469

merged 1 commit into from Dec 22, 2023

Conversation

Eisenwave
Copy link
Contributor

@Eisenwave Eisenwave commented Aug 19, 2023 •
edited

In this paragraph, expression literally refers to when the expresssion grammar term is missing in the syntax (included in the same paragraph):

expressionopt;

It feels more appropriate to style it as a grammarterm due to this strong connection, instead of simple prose.

@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 10, 2023

@Eisenwave Could you please rebase?

@wg21bot wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Nov 10, 2023
@jensmaurer jensmaurer removed the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Dec 22, 2023
@jensmaurer jensmaurer merged commit 08649a5 into cplusplus:main Dec 22, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jensmaurer jensmaurer jensmaurer approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Eisenwave @tkoeppe @jensmaurer @wg21bot