search

sanyaade-g2g-repos / quimeraengine

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

QLineSegment2D Extension 001 #129

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Añade funcionalidad relacionada con transformaciones en el espacio 
bidimensional.

Más información en:

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

Original issue reported on code.google.com by Lince3D@gmail.com on 21 Mar 2011 at 9:23

GoogleCodeExporter commented 9 years ago

Original comment by txan...@gmail.com on 31 May 2011 at 1:57

GoogleCodeExporter commented 9 years ago 
Ready for Review

Original comment by txan...@gmail.com on 13 Jun 2011 at 3:40

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

-Fallo de estilo 1: No se deben mencionar tipos concretos, nombres de clase y 
demás en la documentación, salvo casos especiales. En lugar de decir 
QBaseLineSegment, solemos decir line segment, en plan general, se dice el 
concepto. En la descripción de los parámetros no se nombra el tipo tampoco.

-Fallo de estilo 2: A veces, es mejor poner el concepto del parámetro a secas 
en lugar de un complejo prefijo para indicar el tipo, por ejemplo: matrix, o 
transMatrix en lugar de tm33Matrix.

-El prefijo b- está reservado a variables booleanas.

-Usar métodos de las clases que ya se tienen: Por ejemplo, al restar vectores, 
usar la resta de vectores en lugar de hacerla componente a componente (no se 
consigue mayor rendimiento con ello, ya que al final generarán el mismo 
código). En realidad, en este sentido, hay métodos ya subidos que están mal.

-El punto y coma no es necesario al definir los métodos, detrás de la llave 
de cierre (quita el resto también por favor).

Original comment by Lince3D@gmail.com on 15 Jun 2011 at 9:57

GoogleCodeExporter commented 9 years ago 
Corregido, te vuelvo a mandar el patch

Original comment by txan...@gmail.com on 18 Jun 2011 at 11:37

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

-Documentación de parámetros de todos los métodos: Revisa que coinciden los 
nombres de los parámetros con los escritos en la documentación.

-Documentación de Transform con parámetro de salida: "Receives a 
transformation" = "Receives a transformation matrix".

-Documentación del parámetro de salida de TransformFromPivot: "storages" = 
"stores".

Original comment by Lince3D@gmail.com on 22 Jun 2011 at 7:02