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
LWG Poll 23: P2106R0 Alternative wording for GB315 and GB316 #3774
Conversation
source/algorithms.tex
Outdated
|
||
template<class II1, class II2, class OO> | ||
requires convertible_to<const I1&, II1> && | ||
convertible_to<const I2&, II2> && convertible_to<const O&, OO> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
convertible_to<const I2&, II2> && convertible_to<const O&, OO> | |
convertible_to<const I2&, II2> && convertible_to<const O&, OO> |
https://github.com/cplusplus/draft/pull/3750/files formats it like this. This occurs below too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FWIW, I use that style in #3750 because I'm under the impression that the editors prefer it. It is notably similar to how we indent run-on lines for template parameters.
Let me try to close and reopen this to trigger Travis. |
source/algorithms.tex
Outdated
|
||
template<class II1, class II2, class OO> | ||
requires convertible_to<const I1&, II1> && | ||
convertible_to<const I2&, II2> && convertible_to<const O&, OO> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FWIW, I use that style in #3750 because I'm under the impression that the editors prefer it. It is notably similar to how we indent run-on lines for template parameters.
fc7ed67
to
915c610
Compare
Also fixes NB GB 315 and NB GB 316.
915c610
to
da3b710
Compare
Also fixes NB GB 315 and NB GB 316.
Fixes #3725 .
Fixes cplusplus/nbballot#311.
Fixes cplusplus/nbballot#312.
Open issues: