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
P1456R1 Move-only views #3453
P1456R1 Move-only views #3453
Conversation
Please augment the commit message to add references to the NB ballot comments mentioned above. |
Missing changes to Edit: I see that this is part of a bigger change, and maybe they're taken care of. P1862 particularly seems to add the |
@JohelEGP No, you are right, all views with a |
Yes, "Change [range.transform.view], [range.take.view], [range.common.view], and [range.reverse.view] similarly" in P1456 is missing the range adaptors added by P1035. Are we willing to construe P1456 as providing editorial instructions to make similar changes to all such range adaptors, or shall I submit an LWG issue to make this same change to these four that were missed? |
@CaseyCarter I think the "Proposal" section gives us sufficient motivation to consider this an editorial fix to the wording as moved:
(especially with you as a paper author telling us this was indeed the intent!). |
1ec6391
to
73540cb
Compare
73540cb
to
0a0086f
Compare
Fixes NB GB 277 and FR 281, and partially fixes US 253 and GB 270 (C++20 CD).
take_while_view, drop_view, drop_while_view, and elements_view. These are covered by the direction proposed in the paper, but were inadvertently omitted from the list of things to change in the wording.
0a0086f
to
1dd384a
Compare
destruction explicit in the description of when a type models view.
Fixes #3414.
Fixes cplusplus/papers#219.
Fixes cplusplus/nbballot#273
Partially addresses cplusplus/nbballot#249
Partially addresses cplusplus/nbballot#266
Fixes cplusplus/nbballot#277
Fixes cplusplus/nbballot#289.