QSpaceConversionMatrix

[GoogleCodeExporter]

Representa una matriz que simboliza cambios de sistemas de coordenadas. Añade 
funcionalidad para el cambio a espacio de vista (view space), de proyección 
(projection space) y de mundo (world space).

Más información en:

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

Original issue reported on code.google.com by Lince3D@gmail.com on 19 Dec 2010 at 2:29

[GoogleCodeExporter]

La cojo porque QVector3 Extension 002 tiene dependencia con ella.

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

• Changed state: Started

[GoogleCodeExporter]

Link que puede ayudar:

http://books.google.com/books?id=is6Wc3hKSWIC&pg=PA157&lpg=PA157&dq=frustum+fov+
horizontal+directx&source=bl&ots=_QbNle9Hgp&sig=-NTMEgr10Mfld1tJD-Jqnl1blyE&hl=e
s&ei=rrU-TfTnCIuYhQfYhKnuCg&sa=X&oi=book_result&ct=result&resnum=1&ved=0CBUQ6AEw
AA#v=onepage&q&f=false

Original comment by Lince3D@gmail.com on 25 Jan 2011 at 11:48

[GoogleCodeExporter]

Vaya, lo he visto tarde ese link. Mañana le echo un vistacillo a ver.

Original comment by jwl...@gmail.com on 27 Jan 2011 at 9:51

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-Hace falta revisar qué includes no son necesarios (que a simple vista me 
parece obvio que alguno sobra).

-Documentación de la clase está en español :/

-Documentación del constructor que recibe 4 vf32_q: "param" = "parameter".

-Documentación de operator* y operator*=: Que sepa, no todas las matrices de 
conversión tienen la cuarta columna igual a (0,0,0,1). Además, no son 
"transformation matrices".

-SetWorldSpaceMatrix: ¿El orden de los parámetros es por una razón concreta? 
Me resulta mucho más familiar tener primero la posición, luego la rotación y 
luego la escala.

-SetWorldSpaceMatrix: "[TODO] jwladi: waiting for a constructor in 
QTransformationMatrix that receives the three matrices."

-Documentación de SetProjectionMatrix: "x/y" = "width/height".

-Documentación de SwitchWorldSpaceHand, SwitchViewSpaceHand, 
SwitchProjectionSpaceHand: "convention as default" = "convention by default".

-Cambiar nombres: 
 SwitchWorldSpaceHand = SwitchHandConventionWorldSpaceMatrix
 SwitchViewSpaceHand = SwitchHandConventionViewSpaceMatrix
 SwitchProjectionSpaceHand = SwitchHandConventionProjectionSpaceMatrix

 Son largos, pero más claros, porque se ve fácilmente que hay 3 acciones iguales pero que se diferencian en el tipo de matriz hacia el que van dirigidos. Además el nombre es igual(su comienzo) que el de QTransformationMatrix.

Original comment by Lince3D@gmail.com on 29 Jan 2011 at 6:41

• Changed state: Started

[GoogleCodeExporter]

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

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

-Documentación del constructor que recibe 4 vf32_q: "paramter" = "parameter".

-Documentación de SetWorldSpaceMatrix: ¿Afectará el orden de la 
documentación de los parámetros a lo que escriba Doxygen?

-SetWorldSpaceMatrix: "[TODO] jwladi: waiting for a constructor in 
QTransformationMatrix that receives the three matrices." (recordatorio)

Original comment by Lince3D@gmail.com on 31 Jan 2011 at 11:44

• Changed state: Done-TODO

[GoogleCodeExporter]

Resultado de la revisión (TODOs): Correcta.

-SetWorldSpaceMatrix: No estoy del todo seguro, pero podrías habler hecho:
"*reinterpret_cast<QTransformationMatrix4x4*>(this) = 
QTransformationMatrix4x4(mDisp, mRot, mScale);".

-SwitchHandConventionWorldSpaceMatrix: ¿Por qué usar operadores de 
indireccion? Podría quedar así: "reinterpret_cast<QTransformationMatrix4x4 
*>(this)->SwitchHandConvention();".

Original comment by Lince3D@gmail.com on 30 Mar 2011 at 10:22

• Changed state: Done