sanyaade-g2g-repos / quimeraengine

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

QPoint Extension 001 #180

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Añade funcionalidad para transformaciones de listas de puntos desde un pivote.

Más información en:

http://quimeraengine.googlecode.com/files/QPoint%20Extension%20001.pdf

Original issue reported on code.google.com by Lince3D@gmail.com on 6 Aug 2011 at 8:08

GoogleCodeExporter commented 8 years ago

Original comment by Lince3D@gmail.com on 25 Sep 2011 at 9:35

GoogleCodeExporter commented 8 years ago

Original comment by Lince3D@gmail.com on 27 Sep 2011 at 7:27

GoogleCodeExporter commented 8 years ago

Original comment by Lince3D@gmail.com on 29 Sep 2011 at 9:42

GoogleCodeExporter commented 8 years ago
- Segun "naming conventions" el unsigned int debería empezar por "u" ¿no?. Lo 
digo por nElements (con dudas)
- Documentacion, en general (creo que vale para casi todos los métodos): 
    -- yo igual pondría "the provided pivot" o algo así que expresara que se pasa, no que es arbitrario (con dudas)
    -- Float type: quedamos que sería floating point value o algo parecido
    -- that will be rotated ¿to be rotated _¿around pivot?_? Pasa algo parecido en todas las descripciones de arPoints, supongo que es cuestión de que guste así o no.

- Documentación de RotateFromPivot(QRotationMatrix3x3): parámetro rotation: 
quizá estaría bien decir que es una matriz de rotación. (sobrecargas 3D y 
4D). Lo mismo en el resto de métodos que llevan matriz: scale, translation, 
transformation, etc.

- Documentacion de ScaleFromPivot, sobrecargas para 3 y 4D. En los parámetros 
fScaleX, etc. se debería indicar que son floating point.

Por lo demás bien.

Original comment by jwl...@gmail.com on 30 Sep 2011 at 7:20