QRotationMatrix3x3

[GoogleCodeExporter]

Simboliza una matriz de 3 filas y 3 columnas que representa una rotación en el 
espacio homogéneo. Aporta optimizaciones y conversiones a otras formas de 
representación de rotaciones. Para más información:

http://quimeraengine.googlecode.com/files/QRotationMatrix3x3.pdf

Original issue reported on code.google.com by Lince3D@gmail.com on 12 Dec 2010 at 1:30

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 29 Dec 2010 at 6:44

• Changed state: Started

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 8 Jan 2011 at 5:19

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-Constructor por defecto: Si va a tener lógica propia que no hace uso de la 
clase base, no hace falta llamar al constructor de la clase base ": 
QMatrix3x3()".

-Documentación de constructor por defecto: Sería mejor especificar que se 
inicializa a la identidad.

-Documentación de constructor que acepta OBaseMatrix3x3: No deberíamos 
especificar tipos concretos en las descripciones generales. En la descripción 
del parámetro pones QMatrix3x3 cuando es QRotationMatrix3x3.

-Documentación de constructor que acepta puntero a float_q: "elements 
consecutive" = "consecutive elements".

-Documentación de Reverse: Deja constancia de que es más rápido que el 
genérico.

-Documentación de GetRotation: Al igual que en el constructor, especifica el 
orden de las rotaciones esperado.

-Documentación de GetRotation que acepta QQuaternion: Por curiosidad, ¿a qué 
llamas trace?

-Documentación de ResetToZero: Yo incluso lo dejaría sin documentar, le 
pondría un comentario normal. Sacaremos la documentación de la interfaz 
pñublica pero por si las moscas...

-En la documentación de todos los constructores "genéricos", que aceptan 
QBaseMatrix3x3, float_q*, vf32_q y float_qs, deberíamos opner que aunque se 
permita, se desaconseja su uso por no poder garantizar que se trate de una 
matriz que contenga exclusivamente una rotación.

Original comment by Lince3D@gmail.com on 9 Jan 2011 at 1:35

• Changed state: Started

[GoogleCodeExporter]

Añado:

-_copysign: No tengo muy claro si es una función estandar con ese nombre...

-En GetRotation: "abs(fAngle - QAngle::_Pi) <= QFloat::Epsilon" ¿Por qué no 
usas QFloat::AreEquals?

-GetRotation: "// Due to angles are specified in radians, we converts it to 
degrees" = "// Since angles are specified in radians, we convert it to degrees".

-GetRotation: "fAngle = QAngle::_180;" ¿Siempre se iguala a 180 grados? Creo 
que ahí te quedaste a medias.

-GetRotation: "return;" no entiendo qué hace ahí :S.

-Chorrada línea 119: { debe ir debajo.
-Chorrada línea 104: Dejar una línea en blanco.

Original comment by Lince3D@gmail.com on 9 Jan 2011 at 2:13

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 10 Jan 2011 at 10:58

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

-Constructores generales: "inpredecible behaviour could be happen" = 
"unpredictable behavior could happen".

-"it's faster than default method": Yo pondría "it's faster than base class' 
method".

-GetRotation: "we converts it to degrees" = "we convert it to degrees".

-GetRotation: Es preferible poner un else al final del todo, y quitar ese 
return;.

Original comment by Lince3D@gmail.com on 11 Jan 2011 at 8:26

• Changed state: Done