Skip to content

Scalar multiplication - #242

Merged
Groovounet merged 2 commits into
g-truc:masterfrom
Jaxan:scalars
Oct 5, 2014
Merged

Scalar multiplication#242
Groovounet merged 2 commits into
g-truc:masterfrom
Jaxan:scalars

Conversation

@Jaxan

@Jaxan Jaxan commented Sep 22, 2014

Copy link
Copy Markdown

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
Copy Markdown
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