Skip to content
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 6] P2164R9 views::enumerate #6135

Merged
merged 3 commits into from Mar 8, 2023
Merged

[LWG 6] P2164R9 views::enumerate #6135

merged 3 commits into from Mar 8, 2023

Conversation

burblebee
Copy link
Contributor

@burblebee burblebee commented Feb 18, 2023

Fixes NB FR 14-021, US 48-108 (C++23 CD).

Fixes #6096.
Fixes cplusplus/papers#875
Fixes cplusplus/nbballot#525
Fixes cplusplus/nbballot#416

@burblebee burblebee force-pushed the motions-2023-02-lwg-6 branch 2 times, most recently from 8c472f4 to 2ded891 Compare February 18, 2023 00:24
@burblebee burblebee marked this pull request as ready for review February 18, 2023 00:37
@jensmaurer jensmaurer added this to the post-2023-02 milestone Feb 18, 2023
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/support.tex Outdated Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
@hewillk
Copy link
Contributor

hewillk commented Feb 23, 2023

I did these reviews a week ago, but I didn't realize until now that all my reviews are in PENDING status, which means my reviews haven't been submitted. lol.
If some of my reviews have already been resolved, please ignore them.

@tkoeppe tkoeppe force-pushed the motions-2023-02-lwg-6 branch 3 times, most recently from 66b3e76 to 347232d Compare March 7, 2023 23:42
Editorial changes:
* Added missing "views::" qualification in example
* Added missing "friend" in itemdecls
* Fixed "pos_" to "i.pos_" in iter_move definition
* Removed erroneous and unused default argument "pos = 0"

Fixes NB FR 021, US 108 (C++23 CD).
The comment was attached to the line of the definition of sentinel's
class, but it should appear on the line of the declaration of its
first constructor.
@tkoeppe tkoeppe merged commit a6a8896 into main Mar 8, 2023
4 checks passed
@tkoeppe tkoeppe deleted the motions-2023-02-lwg-6 branch May 10, 2023 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
7 participants