search

dyrks1991 / quimeraengine

Automatically exported from code.google.com/p/quimeraengine
0 stars 0 forks source link

QVector2 Extension 001 #68

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Extiende la funcionalidad de QVector2, integrándola con otras clases.

Más información en:

http://quimeraengine.googlecode.com/files/QVector2-Extension-001.pdf

Original issue reported on code.google.com by Lince3D@gmail.com on 6 Dec 2010 at 1:05

GoogleCodeExporter commented 9 years ago

Original comment by shengp...@gmail.com on 14 Jan 2011 at 11:56

GoogleCodeExporter commented 9 years ago

Original comment by shengp...@gmail.com on 27 Jan 2011 at 1:23

GoogleCodeExporter commented 9 years ago 
Resultado de la revisión: Correcta (no olvides hacer estos pequeños cambios 
primero):

-Documentación operator-: "Vectors negation" = "Vector's negation".

-operator-: Esto es una tontería, pero como tampoco cuesta nada, deja un 
espacio entre el método y el bloque METHODS, en el .cpp.

-operator*=: Puede hacerse inline si se utilizan 2 float_q para guardar las dos 
componentes y asignárselas a *this.

-Documentación de Normalize: "we want store" = "we want to store".

-Documentación de Reverse: "we want store" = "we want to store".

-Documentación del constructor que recibe un QBaseVector2: "The 2D vector  in 
which" sobra un espacio.

Original comment by Lince3D@gmail.com on 27 Jan 2011 at 10:38

GoogleCodeExporter commented 9 years ago 
Hecho!

Original comment by shengp...@gmail.com on 4 Feb 2011 at 2:03

GoogleCodeExporter commented 9 years ago 
Reabierta. Falta por implementar el método DotProductAngle.

Original comment by Lince3D@gmail.com on 11 May 2011 at 9:40

GoogleCodeExporter commented 9 years ago

Original comment by Lince3D@gmail.com on 11 May 2011 at 9:41

GoogleCodeExporter commented 9 years ago 
Owner: anderson_jag

Original comment by anderson...@gmail.com on 13 May 2011 at 4:13

GoogleCodeExporter commented 9 years ago 
Añadido el método DotProductAngle.

Estado: ReadyForReview

Original comment by anderson...@gmail.com on 13 May 2011 at 5:13

GoogleCodeExporter commented 9 years ago 
Resultado de la revisión:

-DotProductAngle: "float_q fLengthProd = QFloat::_0;" y "float_q fAngle = 
QFloat::_0;" ¿Por qué no lo inicializas al valor que va a tener en una línea 
en lugar de asignarlo dos veces?

-DorProductAntle: "QE_ASSERT( (abs(fDot) - QFloat::_1) <= QFloat::Epsilon )" 
puede expresarse con un método comparativo de QFloat.

Original comment by Lince3D@gmail.com on 13 May 2011 at 5:51

GoogleCodeExporter commented 9 years ago 
Corregidos los errores descritos en el comentario anterior.

Estado: ReadyForReview

Original comment by anderson...@gmail.com on 16 May 2011 at 9:31

GoogleCodeExporter commented 9 years ago 
Resultado de la revisión: Correcta.

Original comment by Lince3D@gmail.com on 22 May 2011 at 10:51

GoogleCodeExporter commented 9 years ago 
Subidos los cambios. Revisión: 713

Se ha tenido en cuenta el problema de los valores fuera de
rango para la función 'acos', y se ha añadido una solución
provisional (señalada con [TODO]) dentro de DotProductAngle.

Si es necesrio, cámbiese el estado a Done-TODO.

Original comment by anderson...@gmail.com on 22 May 2011 at 12:58

GoogleCodeExporter commented 9 years ago 
Estado cambiado a: Done-TODO

Razón: Buscar solución a los posibles problemas en DotProductAngle con los 
valores fuera de rango (no detectados si actualmente empleamos funciones de 
comparación de QFloat en lugar de signos de comparación de C++), como puede 
ser 1.0000001f, que puedan llegar a la función 'acos', empleada dentro de 
DotProductAngle.

Original comment by anderson...@gmail.com on 22 May 2011 at 3:57