-
-
Notifications
You must be signed in to change notification settings - Fork 427
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
Making hpx::util::tuple<Ts...> and std::tuple<Ts...> convertible #3961
Conversation
6c54b6a
to
5bdfce8
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!
@sithhell would you have time to look at the build failures? |
94d6844
to
ae99516
Compare
ae99516
to
e917d1c
Compare
This should be ready to go now. |
81791a5
to
dc13ee4
Compare
d08a04d
to
9144237
Compare
9ec5b73
to
cdf91a1
Compare
Fixes #1403 by allowing the use of hpx::util::get with std::tuple and std::get with hpx::util::tuple::get. Same goes for the utilities tuple_size and tuple_element. An additional conversion operator from hpx::util::tuple<Ts..> to std::tuple<Ts..> has been added to make the two types interchangable. Flyby: Making the datastructures unit test only depend on itself and hpx_testing: Adding hpx/traits/is_tuple_like.hpp
…ADL barrier - adding option to disable the integration of hpx::tuple with std::tuple - flyby: fixing default value for HPX_FILESYSTEM_WITH_BOOST_FILESYSTEM_COMPATIBILITY
cdf91a1
to
4f035dc
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 now, thanks! See also @sithhell comments here: http://irclog.cct.lsu.edu/ste~b~~b~ar/2019-09-25#1569431379-1569431420;
Fixes #1403 by allowing the use of hpx::util::get with std::tuple and
std::get with hpx::util::tuple::get. Same goes for the utilities
tuple_size and tuple_element.
An additional conversion operator from hpx::util::tuple<Ts..> to
std::tuple<Ts..> has been added to make the two types interchangable.