Use is_copy_constructible_v - Githubissues

Naios / function2

Improved and configurable drop-in replacement to std::function that supports move only types, multiple overloads and more

http://naios.github.io/function2

Boost Software License 1.0

539 stars 47 forks source link

Use is_copy_constructible_v #41

Closed p4654545 closed 3 years ago

p4654545 commented 3 years ago

gcc can not handle variant<cti::work, ...>. Using is_copy_constructible_v instead of is_copy_constructible<...>::value works around this problem.

@Naios

This solves issue https://github.com/Naios/continuable/issues/37

What was a problem?

Using std::variant<cti::work, ...> is not possible, because of a bug in gcc.

How this PR fixes the problem?

This fix uses is_copy_constructible_v instead of is_copy_constructible<>::value on line 1425.

Check lists (check `x` in `[ ]` of list items)

[ ] Additional Unit Tests were added that test the feature or regression
[ ] Coding style (Clang format was applied)

Additional Comments (if any)

p4654545 commented 3 years ago

I now see the solution should be C++14 and not C++17. sorry.