search

Chinamming / bullet

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

SIMD_2_PI macro definition is wrong in btScalar.h #783

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
It should be enclosed in braces: (btScalar(2.0) * SIMD_PI) . Without them, 
SIMD_DEGS_PER_RAD takes wrong value, this also breaks btDegrees() function. A 
minor issue considering that this function is not used anywhere in engine, so 
this can only affect engine clients. Issue also affects any calculations where 
SIMD_2_PI is a divisor but there's also none in engine or examples (it's always 
at the right side).

Original issue reported on code.google.com by Anton.Br...@gmail.com on 25 Dec 2013 at 4:31

GoogleCodeExporter commented 9 years ago 
Oh, I'm very sorry, it seems that this issue was reported two times already. I 
searched for it before submitting but it seems that I searched only open 
tickets.

Original comment by Anton.Br...@gmail.com on 25 Dec 2013 at 4:40

GoogleCodeExporter commented 9 years ago 
Indeed, I might need to create a new release to avoid repeat reports ;)

Original comment by erwin.coumans on 30 Dec 2013 at 3:35