Skip to content

<ranges> Invalidate the cached position of a view after move - #1931

Merged
Stephan T. Lavavej (StephanTLavavej) merged 11 commits into
microsoft:mainfrom
miscco:cached_position
Aug 6, 2021
Merged

Stephan T. Lavavej (StephanTLavavej) merged 11 commits into
microsoft:mainfrom
miscco:cached_position

Conversation

@miscco

Copy link
Copy Markdown
Contributor

We should invalidate the cached position after a move

Comment thread stl/inc/ranges
Comment thread stl/inc/ranges Outdated
Comment thread tests/std/tests/GH_001914_Cached_position/env.lst Outdated
@CaseyCarter Casey Carter (CaseyCarter) added bug Something isn't working ranges C++20/23 ranges labels May 26, 2021
Comment thread tests/std/tests/GH_001914_Cached_position/env.lst
Comment thread tests/std/tests/GH_001914_Cached_position/env.lst Outdated
Comment thread stl/inc/ranges Outdated
Comment thread stl/inc/ranges Outdated
Comment thread stl/inc/ranges Outdated
Comment thread stl/inc/ranges Outdated
Comment thread stl/inc/ranges
Comment thread stl/inc/ranges Outdated
Comment thread stl/inc/ranges Outdated
Comment thread tests/std/tests/GH_001914_Cached_position/test.cpp Outdated
@CaseyCarter Casey Carter (CaseyCarter) removed their assignment Aug 4, 2021
Comment thread tests/std/tests/GH_001914_Cached_position/env.lst

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good - I'll go ahead and push superficial changes.

Comment thread tests/std/tests/GH_001914_Cached_position/env.lst
Comment thread tests/std/tests/GH_001914_Cached_position/test.cpp
Comment thread tests/std/tests/GH_001914_Cached_position/test.cpp
Comment thread tests/std/tests/GH_001914_Cached_position/test.cpp Outdated
@StephanTLavavej

Copy link
Copy Markdown
Member

FYI Casey Carter (@CaseyCarter) I pushed trivial changes after you approved, to fix the issues that Adam Bucior (@AdamBucior) and I identified.

@StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) merged commit b03f22f into microsoft:main Aug 6, 2021
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for fixing this bug! 🐛 😸 🎉

@miscco
Michael Schellenberger Costa (miscco) deleted the cached_position branch August 6, 2021 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ranges C++20/23 ranges

Projects

None yet

Development

Successfully merging this pull request may close these issues.

<ranges>: Moving from a _Cached_position should invalidate it

4 participants