-
-
Notifications
You must be signed in to change notification settings - Fork 428
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
Adapt partition, partition_copy and stable_partition to C++ 20 #5540
Conversation
07e6066
to
8d3c586
Compare
libs/parallelism/algorithms/include/hpx/parallel/algorithms/partition.hpp
Outdated
Show resolved
Hide resolved
libs/parallelism/algorithms/include/hpx/parallel/algorithms/partition.hpp
Outdated
Show resolved
Hide resolved
libs/parallelism/algorithms/include/hpx/parallel/algorithms/partition.hpp
Outdated
Show resolved
Hide resolved
libs/parallelism/algorithms/include/hpx/parallel/algorithms/partition.hpp
Outdated
Show resolved
Hide resolved
libs/parallelism/algorithms/include/hpx/parallel/algorithms/partition.hpp
Outdated
Show resolved
Hide resolved
libs/parallelism/algorithms/include/hpx/parallel/algorithms/partition.hpp
Outdated
Show resolved
Hide resolved
libs/parallelism/algorithms/include/hpx/parallel/algorithms/partition.hpp
Outdated
Show resolved
Hide resolved
37e6c3d
to
d0d267d
Compare
libs/parallelism/algorithms/include/hpx/parallel/algorithms/partition.hpp
Outdated
Show resolved
Hide resolved
libs/parallelism/algorithms/include/hpx/parallel/algorithms/detail/advance_and_get_distance.hpp
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Are the cosmetic changes to the unrelated algorithms separable? Might be better to create a different PR for those...
Yes I probably shouldn't have done that in this PR, especially given the restructuring done on the master. I'll try and remove those changes. |
d0d267d
to
b852d7a
Compare
… and partition_copy
b852d7a
to
a246d61
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Adapted partition, partition_copy and stable_partition to C++ 20 and added container overloads. (range and sentinel). Added container tests for stable_partition.
Any background context you want to provide?
Issue #4822
Issue #3646
Issue #1668