QCircle

[GoogleCodeExporter]

Representa un círculo en el espacio bidimensional, un conjunto de todos los 
puntos que distan lo mismo de un punto llamado centro.

Más información en:

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

Original issue reported on code.google.com by Lince3D@gmail.com on 31 Aug 2011 at 5:24

[GoogleCodeExporter]

Empiezo

Original comment by txan...@gmail.com on 27 Sep 2011 at 11:04

• Changed state: Started

[GoogleCodeExporter]

ready for review

Original comment by txan...@gmail.com on 14 Oct 2011 at 9:13

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: 

-Por favor, por alinear el código con el refactoring realizado hace poco, 
sustituye:
reinterpret_cast<QBaseOrb<QBaseVector2>&>(*this) = orb;
por:
QBaseOrb<QVector2>::operator=(orb);

-Operator=: Debe recibir un QBaseOrb<QVector2>, no un QBaseOrb<QBaseVector2>.

-UnitCircle: Debe estar inicializada en un CPP a parte, en el pasado tuvimos 
problemas por no hacerlo así.

-Documentación general de parámetro de salida outputCircle: Pones que 
contendrá la traslación, la rotación, la escala... Para ser más correctos, 
habría que decir que contiene el círculo trasladado, rotado o escalado.

-IntersectionPoint: Falta el modificador const al final de la firma de la 
función.

-IntersectionPoint: Utiliza los comparadores de QFloat, por ejemplo: fDistance 
!= QFloat::0 sería QFloat::IsNotZero(fDistance). Lo mismo ocurre al comparar 
vFirstIntersection.x == vSecondIntersection.x.

-IntersectionPoint: No es necesario poner a cero los vectores de salida cuando 
no haya intersección, se debe explicar que no serán modificados.

-IntersectionPoint: ¿No sería suficiente comparar los centros en el paso 5?

Original comment by Lince3D@gmail.com on 19 Oct 2011 at 8:23

• Changed state: Started

[GoogleCodeExporter]

Corregidos los errores indicados.

Original comment by txan...@gmail.com on 20 Oct 2011 at 1:39

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-Operator=: Hay que devolver *this también :)

-Documentación general de parámetro de salida outputCircle: Pones que 
contendrá la traslación, la rotación, la escala... Para ser más correctos, 
habría que decir que contiene el círculo trasladado, rotado o escalado.

-IntersectionPoint: Lo de comparar los centros en el paso 5 lo dije como 
pregunta, supongo que si lo has puesto así es porque has visto que 
efectivamente bastaba con eso.

Original comment by Lince3D@gmail.com on 20 Oct 2011 at 8:41

• Changed state: Started

[GoogleCodeExporter]

Corregidos los problemas detectados.

Original comment by txan...@gmail.com on 22 Oct 2011 at 8:44

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

-Transform y TransformFromPivot: En la documentación de outputCircle pones 
"scaled circle", cuando es "transformed circle".

Original comment by Lince3D@gmail.com on 23 Oct 2011 at 5:03

• Changed state: Done

[GoogleCodeExporter]

Añadidos los últimos cambios indicados en la documentación y subido en la 
revisión 833.

Original comment by txan...@gmail.com on 23 Oct 2011 at 6:09