[expr.prim.lambda] Move operator() return type description #4069
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jensmaurer
added
the
changes requested
jensmaurer
removed
the
changes requested
jensmaurer
reviewed
Jul 4, 2020
reformat operator() declaration description
jensmaurer
approved these changes
Sep 19, 2020
zygoloid
reviewed
Oct 18, 2020
| \item \grammarterm{parameter-declaration-clause} is the \grammarterm{lambda-declarator}'s | ||
| \grammarterm{parameter-declaration-clause}, | ||
|
|
||
| \item return type is \tcode{auto}, |
There was a problem hiding this comment.
I think that this isn't the return type, so much as it is the type in the decl-specifier-seq. (The declared return type is determined by the trailing-return-type if any, and the actual return type is what we deduce. Per [dcl.spec.auto.general]p3, the auto is just something that can appear in the decl-specifier-seq; it's not really a return type at all.) So:
Suggested change
| \item return type is \tcode{auto}, | |
| \item \grammarterm{decl-specifier-seq} is the \grammarterm{lambda-declarator}'s \grammarterm{decl-specifier-seq} followed by \keyword{auto}, except that \keyword{constexpr} is also included if [...] |
Also... should we say something about the attribute-specifier-seq?
zygoloid
added
the
changes requested
|
@languagelawyer , this needs a bit more work. |
wg21bot
added
the
needs rebase
|
Can you please rebase, and address the open comments? |
|
@languagelawyer, ping. |
|
Please reopen if you would like to continue with this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.