Skip to content
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

In cppx/meta/tuple.hpp, change T::size() to std::tuple_size<T>::value #53

Closed
drec357 opened this issue Nov 28, 2018 · 0 comments
Closed

Comments

@drec357
Copy link

drec357 commented Nov 28, 2018

Small issue, simple fix: in the tuple.hpp file in your reflection library, you have specified std::tuple_size<...> specializations in an apparent effort to generalize your tuple algorithms so they can be used on e.g. std::tuples in addition to your reflected_tuples or whatever; however in the two tuple_for_each_recursive definitions, T::size() is still used in the enable_if_t<...> instead of std::tuple_size<T>::value, making it not as general as it could be -- it fails with std::tuples right now.

@drec357 drec357 closed this as completed Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant