Bart/bullet3
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

documentation/parameter name fix for btMatrix3x3::setEulerZYX. There is no change in behaviour, no compatibility issues.

Browse Source 
Thanks Tully Foote for the patch.
http://code.google.com/p/bullet/issues/detail?id=157
This commit is contained in:
erwin.coumans
2008-12-02 22:27:26 +00:00
parent 76b3e44117
commit 23bb0d94e6
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/LinearMath/btMatrix3x3.h
View File

@@ -159,16 +159,15 @@ class btMatrix3x3 {
} }
/** @brief Set the matrix from euler angles YPR around ZYX axes /** @brief Set the matrix from euler angles YPR around ZYX axes
* @param eulerX Roll about X axis
* @param eulerY Pitch around Y axis
* @param eulerZ Yaw aboud Z axis * @param eulerZ Yaw aboud Z axis
* @param eulerY Pitch around Y axis
* @param eulerX Roll about X axis
* *
* These angles are used to produce a rotation matrix. The euler * These angles are used to produce a rotation matrix. The euler
* angles are applied in ZYX order. I.e a vector is first rotated * angles are applied in ZYX order. I.e a vector is first rotated
* about X then Y and then Z * about X then Y and then Z
**/ **/
void setEulerZYX(btScalar eulerX,btScalar eulerY,btScalar eulerZ) { void setEulerZYX(btScalar eulerZ, btScalar eulerY,btScalar eulerX) {
///@todo proposed to reverse this since it's labeled zyx but takes arguments xyz and it will match all other parts of the code
btScalar ci ( btCos(eulerX)); btScalar ci ( btCos(eulerX));
btScalar cj ( btCos(eulerY)); btScalar cj ( btCos(eulerY));
btScalar ch ( btCos(eulerZ)); btScalar ch ( btCos(eulerZ));