Skip to content

Implement ranges::move_backward - #1053

Merged
Casey Carter (CaseyCarter) merged 7 commits into
microsoft:masterfrom
CaseyCarter:move_backward
Jul 28, 2020
Merged

Implement ranges::move_backward#1053
Casey Carter (CaseyCarter) merged 7 commits into
microsoft:masterfrom
CaseyCarter:move_backward

Conversation

@CaseyCarter

Copy link
Copy Markdown
Contributor

No description provided.

@CaseyCarter Casey Carter (CaseyCarter) added cxx20 C++20 feature ranges C++20/23 ranges labels Jul 17, 2020
@CaseyCarter
Casey Carter (CaseyCarter) requested a review from a team as a code owner July 17, 2020 18:08
Comment thread tests/std/tests/P0896R4_ranges_alg_move_backward/test.cpp
Comment thread tests/std/tests/P0896R4_ranges_alg_move_backward/test.cpp Outdated
Comment thread tests/std/tests/P0896R4_ranges_alg_move_backward/test.cpp Outdated
Comment thread tests/std/tests/P0896R4_ranges_alg_move_backward/test.cpp Outdated
Comment thread stl/inc/algorithm Outdated
Comment thread tests/std/tests/P0896R4_ranges_alg_move_backward/test.cpp Outdated
Comment thread tests/std/tests/P0896R4_ranges_alg_move_backward/test.cpp Outdated

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, requesting changes only for the namespace-scope static.

Comment thread stl/inc/algorithm Outdated
Comment thread tests/std/tests/P0896R4_ranges_alg_move_backward/test.cpp Outdated
Co-authored-by: Stephan T. Lavavej <stl@nuwen.net>
Comment thread tests/std/tests/P0896R4_ranges_alg_move_backward/test.cpp
Co-authored-by: mnatsuhara <46756417+mnatsuhara@users.noreply.github.com>
@CaseyCarter

Copy link
Copy Markdown
Contributor Author

I appreciate this contribution, but I can't help but feel that we're moving backward when we should be moving forward. 🤔

@CaseyCarter Casey Carter (CaseyCarter) removed their assignment Jul 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cxx20 C++20 feature ranges C++20/23 ranges

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants