Creación de constructores de copia para clases y clases base en Math

[GoogleCodeExporter]

Cada clase X de Math debe tener un constructor de copia, que recibirá un tipo 
X, y otro constructor que recibe un tipo de clase base de X. 

Importante: Hay que crear un unit test por cada constructor que no existiera.

Original issue reported on code.google.com by Lince3D@gmail.com on 3 May 2012 at 8:58

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

Pillada

Original comment by borderpa...@gmail.com on 30 May 2012 at 5:29

• Changed state: Started

[GoogleCodeExporter]

-Constructores de copia y de tipo base implementados. Ojo al revisar sobre todo 
los tipos de plantilla, no tengo muy claro si la funcionalidad que se pedía 
era exactamente esta para esos tipos (sobre todo los geométricos).
-Unit tests actualizados.

Original comment by borderpa...@gmail.com on 11 Jul 2012 at 4:26

• Changed state: ReadyForReview

[GoogleCodeExporter]

-Constructores de copia y de tipo base implementados. Ojo al revisar sobre todo 
los tipos de plantilla, no tengo muy claro si la funcionalidad que se pedía 
era exactamente esta para esos tipos (sobre todo los geométricos).
-Unit tests actualizados.

Original comment by borderpa...@gmail.com on 11 Jul 2012 at 4:26

[GoogleCodeExporter]

Resultado de la revisión:

-Constructor de copia de QBaseDualQuaternion: Por favor cada sentencia en una 
línea.

-Documentación de constructor de copia de QBaseDualQuaternion, QBaseMatrix2x2, 
QBaseMatrix3x3, QBaseMatrix3x4, QBaseMatrix4x3, QBaseMatrix4x4, 
QBaseQuaternion, QBaseVector2, QBaseVector3, QBaseVector4: "Copy attributes 
from" = "Copies attributes from".

-Constructor de copia de QCircle: El parámetro de tipo QCircle no necesita 
parámetros de plantilla (QVector2).

-Documentación constructor de copia con el tipo base de QDualQuaternion: Sobra 
un punto y final.

-Constructor de copia de QLineSegment2D: Debe recibir un QLineSegment2D.

-Constructor de copia de QLineSegment3D: Debe recibir un QLineSegment3D.

-Documentación de constructores de QSpaceConversionMatrix: En vez de nombrar 
el parámetro "space" lo llamaría "matrix" o "spaceConversionMatrix".

-Constructor de copia de QTranslationMAtrix y QTransformationMatrix: La clase 
del parámetro debe incluir el parámetro de plantilla <MatrixType>.

-Constructor de tipo base QTriangle2D: Usa QBaseTriangle<QVector2> en lugar de 
QBaseTriangle2D.

-Los tests con nombre "CopyIsIndependent" no son necesarios. Dado el diseño de 
las clases de Math, esos tests siempre pasarán por mucho que se cambie la 
implementación de las clases.

-Test de constructor de copia de QDualQuaternion repetido: Constructor2 y 
Constructor3, ambos reciben un QDualQuaternion, cuando deberían recibir uno un 
QDualQuaternion y el otro QBaseDualQuaternion.

-Test de constructor de copia de QQuaternion repetido: Constructor2 y 
Constructor3, ambos reciben un QQuaternion, cuando deberían recibir uno un 
QQuaternion y el otro QBaseQuaternion.

-Falta test para el constructor con tipo base para QVector3.

Original comment by Lince3D@gmail.com on 15 Jul 2012 at 9:17

• Changed state: Started

[GoogleCodeExporter]

Correcciones completadas.

Original comment by borderpa...@gmail.com on 19 Jul 2012 at 6:57

• Changed state: ReadyForReview

[GoogleCodeExporter]

Original comment by Lince3D@gmail.com on 19 Jul 2012 at 9:20

• Changed state: Done

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

Original comment by Lince3D@gmail.com on 19 Jul 2012 at 9:23