Skip to content

Conversation

Jaxan
Copy link

@Jaxan Jaxan commented Sep 22, 2014

One of the annoying parts of glsl is that the following doesn't compile:

glm::vec3 v(1, 2, 3);
glm::vec3 w = 2 * v; // error :(

You correclty mirrored this in glm, which is good. But in our old codebase we did these kind of multiplications, because they are just too convenient ;-). Instead of changing our code, I added these multiplications.

Groovounet added a commit that referenced this pull request Oct 5, 2014
@Groovounet Groovounet merged commit 36af05b into g-truc:master Oct 5, 2014
@Groovounet
Copy link
Member

I need to make a small update for C++98 support but that look fine.

Thanks for contributing!
Christophe

@Groovounet Groovounet added the bug label Oct 5, 2014
@Groovounet Groovounet added this to the GLM 0.9.6 milestone Oct 5, 2014
@Groovounet Groovounet self-assigned this Oct 5, 2014
@Jaxan Jaxan deleted the scalars branch October 5, 2014 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants