QTranslationMatrix

[GoogleCodeExporter]

Simboliza una matriz de 4 filas y 3 ó 4 columnas, que representa una 
translación en el espacio tridimensional. La razón de que sea una plantilla 
es que la implementación para 3 o para 4 columnas es la misma.

Más información en:

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

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

[GoogleCodeExporter]

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

• Changed state: Started

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 12 Jan 2011 at 8:18

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-"displazament": Reemplazar en todo el fichero por "displacement" (los hay con 
minúscula y con mayúscula).

-"behaviour": Reemplazar en todo el fichero por "behavior".

-Documentación de constructor que acepta un puntero: "es" = "is".

-Documentación de operator*: "Since both are translation matrix" = "Since both 
are translation matrices".

-Documentación de operator* y operator*=: "or one of each type" esto no puede 
darse debido a que el parámetro de plantilla actúa sobre el tipo del 
parámetro, por lo que ambos siempre serán del mismo tipo.

-Documentación de operator*: En returns "with the same template" = "with the 
same template parameter".

-Documentación de Reverse: "the inverse is composed by the opposite" -> "by" = 
"of". "which defines de" = "which defines the".

-¿Getscale?: Revisa esos métodos, es un fallo de copypaste.

Original comment by Lince3D@gmail.com on 13 Jan 2011 at 8:49

• Changed state: Started

[GoogleCodeExporter]

Añado:

-operator* y operator*=: El nombre de su parámetro de plantilla debe ser 
diferente del del parámetro de plantilla de la clase, que podría llamarse 
BaseMatrixType. Haciendo este cambio entonces el comentario "or one of each 
type" sí tendría sentido.

-Hay que definir después de la declaración de la clase 2 typedefs: 
QTranslationMatrix4x3 y QTranslationMatrix4x4, el primero usando QMatrix4x3 
como parámetro de plantilla y en el segundo QMatrix4x4.

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

[GoogleCodeExporter]

Esto lo envié hace un par de días, se me debió pasar cambiar el estado.

Original comment by jwl...@gmail.com on 16 Jan 2011 at 8:11

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

-Documentación de operator*: "Since both are translation matrix" = "Since both 
are translation matrices".

-CONSTANTS INITIALIZATION: Falta espacio a la izquierda y alargar el 
"subrayado".

Original comment by Lince3D@gmail.com on 21 Jan 2011 at 5:58

• Changed state: Done