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.

Sign up for GitHub

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

Jump to bottom

Corrections constructor #3

Open

sv-91 wants to merge 1 commit into AlexeyAB:master from sv-91:master

sv-91 commented Oct 11, 2019

2 correction:

std::forward
make_safe instead constructor


          Corrections constructor

9bc1644

Owner

AlexeyAB commented Oct 11, 2019

A few words why is it better to use make_safe instead constructor?

Author

sv-91 commented Oct 11, 2019

Template parameter in the constructor sometimes causes a compilation error. Since the compiler cannot distinguish 2 constructors.

template
A(B &&)

and

A(const A&)

https://mpark.github.io/programming/2014/06/07/beware-of-perfect-forwarding-constructors/

Instead of this method, you can try sfinae

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet