elefher / glmatrix

Automatically exported from code.google.com/p/glmatrix
0 stars 0 forks source link

mat4.lookat bug #24

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
In mat4.lookat(...),  "return mat4.identity()" should be "return 
mat4.identity(dest)".

In general I think that there you need to make a clearer separation of 
destructive vs non destructive operations eg

function mat4.setIdentity(dest) { ..... return undefined;}

 and 

function mat4.identity() {var dest=mat4.create(); mat4.setIdentity(dest); 
return dest; } 

Also I notice there isn't a quat4.create() method at the moment.

And +1 for unit testing :-)

Original issue reported on code.google.com by Drew.Whitehouse on 24 Jun 2010 at 1:59

GoogleCodeExporter commented 8 years ago

Original comment by Tojiro@gmail.com on 2 Jul 2010 at 4:44

GoogleCodeExporter commented 8 years ago

Original comment by Tojiro@gmail.com on 27 Feb 2011 at 6:18