Skip to content

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
@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.

@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