QTriangle2D

[GoogleCodeExporter]

Representa un triángulo bidimensional. La mayor parte de la funcionalidad 
aportada a la lcase base está relacionada con transformaciones.

Más información en:

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

Original issue reported on code.google.com by Lince3D@gmail.com on 2 Apr 2011 at 1:05

[GoogleCodeExporter]

Original comment by txan...@gmail.com on 8 May 2011 at 7:16

[GoogleCodeExporter]

Original comment by txan...@gmail.com on 8 May 2011 at 8:48

• Changed state: Started

[GoogleCodeExporter]

Ready for review

Pendiente queda el constructor con parámetros vf32_q. Está codificado pero no 
se recomienda su uso hasta que se revise el funcionamiento correcto de este 
tipo de dato.

Original comment by txan...@gmail.com on 8 Jul 2011 at 2:49

• Changed state: ReadyForReview

[GoogleCodeExporter]

Funcionalidad no implementada aún en su totalidad.

Original comment by Lince3D@gmail.com on 25 Jul 2011 at 7:20

• Changed state: Started

[GoogleCodeExporter]

Ready for Review. Añadidos ortocentro y circuncentro

Original comment by txan...@gmail.com on 21 Aug 2011 at 9:25

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-Documentación de constructor que recibe 3 punteros: "Each pointer must have" 
= "Each pointer references to an array which has".

-Por mantener el estilo: Los ampersand (&) que denotan referencia-a-tipo y los 
asteriscos que denotan puntero-a-tipo, deberían ir pegados a la derecha del 
tipo (tipo* x, tipo& x. Cuando indiquen desreferencia irán pegados a la 
izquierda de la variable. Nunca separados de ambos.

-Transform: Comentarios con STEP1 sin ningún paso posterior igual podrían 
quitarse para evitar confusión.

-Optimización general: ¿No crees que se podría optimizar todo si en lugar de 
usar variables auxiliares para los vértices utilizaras los propios vértices 
de los triángulos para almacenar los resultados?

-Documentación de TransformFromPivot: Has escrito que el origen de la 
transformación es el origen de coordenadas.

-GetCircumcenter: Has utilizado un 2, en lugar de QFloat::_2. Ya que estás, 
multiplica por QFloat::_0_5 en lugar de dividir, es ligeramente más rápido.

-GetOrthocenter: Has utilizado 1 en lugar de QFloat::_1.

-Documentación de GetCircumcenter y GetOrthocenter: Falta un punto al final.

-Aquellos métodos que no consideres que deban ser inline puedes llevarlos a un 
CPP.

Original comment by Lince3D@gmail.com on 22 Aug 2011 at 8:15

• Changed state: Started

[GoogleCodeExporter]

Corregido.

Original comment by txan...@gmail.com on 25 Aug 2011 at 5:43

[GoogleCodeExporter]

Actualizo el estado a ready for review.

Original comment by txan...@gmail.com on 25 Aug 2011 at 5:59

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

-Los asteriscos de los punteros van pegados al tipo, no a la variable.

-Documentación de TransformFromPivot: "coordinate vector type" creo que 
podría quitarse.

-No es necesario utilizar la variable auxiliar vPivot en ningún sitio, ya que 
v2Pivot no se modifica en ningún momento.

Original comment by Lince3D@gmail.com on 25 Aug 2011 at 8:26

• Changed state: Done

[GoogleCodeExporter]

Subido en la revisión 798.

También se han corregido los operadores + y - de QVector2. Estos métodos 
aceptaban un parámetro de tipo QVector2, cuando en realidad deben aceptar un 
tipo QBaseVector2 (según lo que hablamos el viernes)
Aprovechando esto, se ha corregido también los métodos TransformFromPivot de 
QLineSegment2D que aceptan como parámetro una QTransformationMatrix3x3 porque 
estaban afectados por el error de QVector2 y obligaban a declarar una variable 
intermedia innecesaria para poder trabajar con el pivote.

Original comment by txan...@gmail.com on 28 Aug 2011 at 12:08