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

Add directive to force default constructors #366

Closed
patnashev opened this issue Jul 24, 2015 · 3 comments
Closed

Add directive to force default constructors #366

patnashev opened this issue Jul 24, 2015 · 3 comments
Assignees
Milestone

Comments

@patnashev
Copy link

If constructors and assignment operator are defined as default:

GLM_FUNC_DECL tvec2() = default;
GLM_FUNC_DECL tvec2(tvec2<T, P> const & v) = default;
GLM_FUNC_DECL tvec2<T, P>& operator=(tvec2<T, P> const & v) = default;

then tvec2 can be used in unions. Implicitly-defined constructors provide no functionality but make the classes non-trivial.

@plasmacel
Copy link

it's not a bad idea

@Groovounet
Copy link
Member

I agree, I like it.

@Groovounet
Copy link
Member

This feature has been in master branch for GLM 0.9.7 release.

Thanks,
Christophe

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

No branches or pull requests

3 participants