sanyaade-g2g-repos / quimeraengine

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

Reemplazar reinterpret_casts por métodos As #225

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Consiste en buscar los usos de reinterpret_cast en Math y reemplazarlos por el 
uso de los métodos As de la siguiente manera:

reinterpret_cast<QRay2D&>(ray).GetPoint();
--->
ray.As<QRay2D>().GetPoint();

Original issue reported on code.google.com by Lince3D@gmail.com on 11 Dec 2011 at 10:00

GoogleCodeExporter commented 8 years ago

Original comment by jwl...@gmail.com on 12 Dec 2011 at 6:02

GoogleCodeExporter commented 8 years ago

Original comment by jwl...@gmail.com on 20 Dec 2011 at 9:57

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

-TODOs: Te iba a mandar cómo hacerlos pero caundo estuve compilando había 
errores de ambigüedad con otros operator*. Ambos métodos, donde tienes los 2 
TODOs, deben ser eliminados.

QTranslationMatrix: No compila a menos que se añadan los includes para 
QBaseVector3 y QBaseVector4.

QTranslateMatrix: Los 2 Operator* que reciben QTranslationMatrix fallan, hay 
que corregir una línea poniendo "ProductOperatorImp(m, outMatrix);".

QTranslationMatrix: Los 2 Operator*= que reciben un QTranslationMatrix  están 
mal, falta el parámetro m en "ProductAssignationOperatorImp(m);";.

Son fallos que no son de la tarea pero me he dado cuenta gracias a ella y son 
instantáneos de corregir. Por lo demás perfecto.

Original comment by Lince3D@gmail.com on 23 Dec 2011 at 10:16