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

[lex.operators] Add co_await to the list #3645

Closed
wants to merge 1 commit into from
Closed

[lex.operators] Add co_await to the list #3645

wants to merge 1 commit into from

Conversation

wreien
Copy link

@wreien wreien commented Jan 22, 2020

co_await is missing from the list of preprocessing-op-or-punc. Since it is an operator like new and delete (see [over.oper]), it should be here, otherwise it would be an identifier in phases 3 through 6. This is a clear inconsistency.

(I'm not certain about how to layout the table, as co_await is almost too big to fit, but the existing patterns should be maintained.)

`co_await` is missing from the list of _preprocessing-op-or-punc_. Since it is an operator like `new` and `delete` (see [over.oper]), it should be here, otherwise it would be an identifier in phases 3 through 6. This is a clear inconsistency.

(I'm not certain about how to layout the table, as `co_await` is almost too big to fit, but the existing patterns should be maintained.)
@jensmaurer
Copy link
Member

I note that the table also does not contain "throw", which appears in a throw-expression.

What exactly breaks if co_await and throw are not present in preprocessing-op-or-punc?
Just some #if conditionals? If so, it seems perfectly reasonable to omit co_await and throw from the list, because those have no reason to appear in an #if.

@jensmaurer jensmaurer added the decision-required A decision of the editorial group (or the Project Editor) is required. label Jan 23, 2020
@jensmaurer
Copy link
Member

Editorial meeting: Not editorial. Covered by CWG 189 and CWG 1901.

@jensmaurer jensmaurer closed this Feb 10, 2020
@jensmaurer jensmaurer removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Feb 10, 2020
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.

None yet

2 participants