QQuaternion Extension 001

[GoogleCodeExporter]

Ampliación de la funcionalidad de QQuaternion, añadiendo operaciones con 
otras clases.

Más información en:

http://quimeraengine.googlecode.com/files/QQuaternion%20Extension%20001.pdf

Original issue reported on code.google.com by Lince3D@gmail.com on 15 Dec 2010 at 8:33

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 2 Feb 2011 at 8:15

• Changed state: Started

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 11 Feb 2011 at 9:22

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-operator*= que recibe un QBaseVector4: Falta un & en el valor devuelto y un *= 
en lugar de un *.

-Constructor de QBaseQuaternion que recibe un QBaseQuaternion: Más bien es un 
constructor de copia, no sé por qué lo hice en su día, cuando no estamos 
haciendo ningún constructor de copia. Bórralo por favor.

-Sería interesante que en adelante no especificaras en la documentación la 
dimensión de las matrices cuando se trate de matrices de transformación, ya 
que así es más flexible de cara al futuro, no hay que cambiarlo si esto 
cambiara.

-"// [TODO] Thund: Use UnitReverse, when it exists.": Por favor, cambia 
Conjugate por UnitReverse.

-ToAxisAngle: "fAngle = 2 * acos(this->w);", cambiar 2 por QFloat::_2.

-Documentación de DotProductAngle: No calcula el ángulo, sino el coseno del 
ángulo (siempre y cuando ambos cuaterniones estén normalizados).

Original comment by Lince3D@gmail.com on 14 Feb 2011 at 8:59

• Changed state: Started

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 14 Feb 2011 at 10:30

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

Original comment by Lince3D@gmail.com on 17 Feb 2011 at 12:01

• Changed state: Done