Skip to content

[expr.prim.lambda] Avoid decl-specifier-seq in lambda-declarator CWG2509 #2338

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
Closed
wants to merge 1 commit into from
Closed

[expr.prim.lambda] Avoid decl-specifier-seq in lambda-declarator CWG2509 #2338

wants to merge 1 commit into from

Conversation

jensmaurer
Copy link
Member

Using a decl-specifier-seq pulls in a lot of undesirable
connotations, plus the following text only refers to
individual decl-specifiers (by name), never to the entire
decl-specifier-seq.

Therefore, introduce a new grammar non-terminal
'lambda-specifier-seq'.

Fixes #2333.

@jensmaurer jensmaurer added the cwg Issue must be reviewed by CWG. label Sep 22, 2018
@jensmaurer jensmaurer mentioned this pull request Sep 22, 2018
Closed
@timsong-cpp
Copy link
Contributor

We need something that says each lambda-specifier can only appear at most once in a complete lambda-specifier-seq.

@jensmaurer
Copy link
Member Author

@timsong-cpp: Right, fixed.

@jensmaurer
Copy link
Member Author

jensmaurer commented Nov 7, 2018 •
edited
Loading

CWG in San Diego: This will become a core issue.

https://wiki.edg.com/bin/view/Wg21sandiego2018/CoreWorkingGroup#Editorial_issue_2338_expr_prim_l

@jensmaurer jensmaurer added the not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking. label Nov 7, 2018
@jensmaurer jensmaurer added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Nov 26, 2018
@jensmaurer jensmaurer removed the needs rebase The pull request needs a git rebase to resolve merge conflicts. label May 28, 2019
@wg21bot wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Jun 15, 2021
@jensmaurer jensmaurer removed the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Jun 15, 2021
@timsong-cpp
Copy link
Contributor

This now also needs something that disallows having both constexpr and consteval.

@jensmaurer jensmaurer added the changes requested Changes to the wording or approach have been requested and not yet applied. label Jun 15, 2021
@wg21bot wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Oct 24, 2021
@jensmaurer
[expr.prim.lambda] Avoid decl-specifier-seq in lambda-declarator
74272e3 
Using a decl-specifier-seq pulls in a lot of undesirable
connotations, plus the following text only refers to
individual decl-specifiers (by name), never to the entire
decl-specifier-seq.

Therefore, introduce a new grammar non-terminal
'lambda-specifier-seq'.
@jensmaurer jensmaurer removed needs rebase The pull request needs a git rebase to resolve merge conflicts. changes requested Changes to the wording or approach have been requested and not yet applied. labels Oct 25, 2021
@jensmaurer
Copy link
Member Author

Pinged Mike Miller to create a core issue, as per San Diego CWG consensus.

@jensmaurer
Copy link
Member Author

CWG telecon 2021-12-09: Moved CWG2509 to Ready.

@jensmaurer jensmaurer changed the title [expr.prim.lambda] Avoid decl-specifier-seq in lambda-declarator [expr.prim.lambda] Avoid decl-specifier-seq in lambda-declarator CWG2509 Dec 9, 2021
@jensmaurer jensmaurer mentioned this pull request Feb 11, 2022
Merged
@jensmaurer jensmaurer deleted the b34 branch February 16, 2022 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cwg Issue must be reviewed by CWG. not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jensmaurer @timsong-cpp @wg21bot