search

dyrks1991 / quimeraengine

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

QBaseRay #63

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Representa un "rayo" en el espacio tridimensional, esto es, un punto del cual 
parte una normal o dirección.

Más información en:

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

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

GoogleCodeExporter commented 8 years ago

Original comment by jwl...@gmail.com on 8 Dec 2010 at 11:00

GoogleCodeExporter commented 8 years ago

Original comment by jwl...@gmail.com on 8 Dec 2010 at 11:42

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

- Constructor quer acepta 2 vectores: No hacen falta las variables auxiliares, 
usa this->Direction = QVector3(qDirection).

- Mismo constructor: ¿Por qué los parámetros tienen refijo "q"? ¿No sería 
mejor una "v"?

- [TODO]: Hay que acostumbrarse a escribir el nombre de quién los puso.

- La tarea no pasará la revisión hasta que las dependencias estén resueltas 
y el código implementado, salvo excepciones de interbloqueo.

Original comment by Lince3D@gmail.com on 8 Dec 2010 at 8:32

GoogleCodeExporter commented 8 years ago

Original comment by jwl...@gmail.com on 8 Dec 2010 at 8:59

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

-En la documentación de los parámetros sigue poniendo qPoint.
-En cuanto esté implementado el QVector4 lo puedes actualizar y subir.

Original comment by Lince3D@gmail.com on 9 Dec 2010 at 5:21

GoogleCodeExporter commented 8 years ago 
Corregidos los parámetros.
La paso a estado Hinded. 

[TODO] jwladi: Replace QBaseVector4.h by QVector4.h when it exists, and all 
references to QBaseVector4 in code by QVector4.

Original comment by jwl...@gmail.com on 9 Dec 2010 at 6:45

GoogleCodeExporter commented 8 years ago

Original comment by jwl...@gmail.com on 28 Dec 2010 at 3:34

GoogleCodeExporter commented 8 years ago

Original comment by jwl...@gmail.com on 28 Dec 2010 at 7:50

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

-Elminina estas inclusiones, ya que vienen dadas en los vectors:

#include "QFloat.h"
#include "QVF32.h"

Original comment by Lince3D@gmail.com on 30 Dec 2010 at 12:51