[stmt.return], [stmt.return.coroutine] update notes re implicit move #5783
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: I'd like a core opinion for this, would you have one or could recommend someone? |
jensmaurer
requested changes
Aug 23, 2022
source/statements.tex
Outdated
| A copy operation associated with a \tcode{return} statement can be elided or | ||
| converted to a move operation if an automatic storage duration variable is returned\iref{class.copy.elision}. | ||
| The copy operation associated with a \tcode{return} statement can be elided or | ||
| converted to a move operation if an implicitly movable entity\iref{class.copy.elision} is returned. |
There was a problem hiding this comment.
(yes, pre-existing, but while we're touching this)
It seems unclear whether the "if" applies to just the "converted to a move operation" part or to the entire "can be elised or converted to a move operation".
Quuxplusone
force-pushed
the
implicit-move-notes
branch
from
685d884
to
f4ea250
Compare
Quuxplusone
commented
Sep 22, 2022
| @@ -913,6 +913,10 @@ | |||
| \item | |||
| If the operand is a \grammarterm{braced-init-list} or an expression of non-\keyword{void} type, | |||
| \placeholder{S} is \placeholder{p}\tcode{.return_value(}\grammarterm{expr-or-braced-init-list}{}\tcode{)}. | |||
| \begin{note} | |||
| The overload resolution of \placeholder{S} is affected | |||
These notes probably should have been changed when P1825 was adopted.
Quuxplusone
force-pushed
the
implicit-move-notes
branch
from
f4ea250
to
2397ad9
Compare
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.
These notes probably should have been changed when P1825 was adopted.