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

Adopt LWG-3533 Make base() const & consistent across iterator wrappers that support input_iterators #1993

Merged
merged 8 commits into from
Jul 30, 2021

Conversation

miscco
Copy link
Contributor

@miscco miscco commented Jun 9, 2021

A I failed to get element_view to accept my clumsy tests.

Does it add value?

I did split out transform_view::iterator tests

Addresses microsoft#1965
@miscco miscco requested a review from a team as a code owner June 9, 2021 13:21
@CaseyCarter CaseyCarter changed the title Adopt LWG-3546 Adopt LWG-3533 Jun 9, 2021
@StephanTLavavej StephanTLavavej added the LWG Library Working Group issue label Jun 10, 2021
@StephanTLavavej StephanTLavavej changed the title Adopt LWG-3533 Adopt LWG-3533 Make base() const & consistent across iterator wrappers that support input_iterators Jun 10, 2021
@CaseyCarter CaseyCarter mentioned this pull request Jun 10, 2021
36 tasks
stl/inc/ranges Outdated Show resolved Hide resolved
stl/inc/ranges Outdated Show resolved Hide resolved
stl/inc/ranges Outdated Show resolved Hide resolved
stl/inc/ranges Outdated Show resolved Hide resolved
stl/inc/ranges Outdated Show resolved Hide resolved
tests/std/tests/P0896R4_views_transform_iterator/test.cpp Outdated Show resolved Hide resolved
tests/std/tests/P0896R4_views_transform_iterator/test.cpp Outdated Show resolved Hide resolved
tests/std/tests/P0896R4_views_transform_iterator/test.cpp Outdated Show resolved Hide resolved
tests/std/tests/P0896R4_views_transform_iterator/test.cpp Outdated Show resolved Hide resolved
tests/std/tests/P0896R4_views_filter_iterator/test.cpp Outdated Show resolved Hide resolved
tests/std/tests/P0896R4_views_transform_iterator/test.cpp Outdated Show resolved Hide resolved
@CaseyCarter CaseyCarter added the ranges C++20/23 ranges label Jun 22, 2021
@CaseyCarter CaseyCarter removed their assignment Jul 15, 2021
Copy link
Member

@StephanTLavavej StephanTLavavej left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, I'll push the dropped noexcepts.

stl/inc/ranges Outdated Show resolved Hide resolved
tests/std/tests/P0896R4_views_filter_iterator/test.cpp Outdated Show resolved Hide resolved
@StephanTLavavej StephanTLavavej removed their assignment Jul 27, 2021
@StephanTLavavej
Copy link
Member

FYI @CaseyCarter, I pushed simple changes after you approved.

@StephanTLavavej StephanTLavavej self-assigned this Jul 29, 2021
@StephanTLavavej StephanTLavavej merged commit a719960 into microsoft:main Jul 30, 2021
@StephanTLavavej
Copy link
Member

Thanks for implementing this LWG issue! 😸 ✔️ 🎉

@miscco miscco deleted the LWG-3533-iterator-base branch August 3, 2021 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LWG Library Working Group issue ranges C++20/23 ranges
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants