Cambio métodos Transform de QLineSegment3D y QLineSegment2D

[GoogleCodeExporter]

Van a incorporarse una serie de funcionalidades que obligan a cambiar el 
diseño de determinadas clases. En este caso, se trata de QLineSegment2D y 
QLineSegment3D, cuyo método Transform actuará ahora aplicando 
transformaciones usando el origen de coordenadas como pivote y no un extremo de 
la recta. 
El método TransformFromCenter se llamará ahora TransformFromPivot y todas sus 
sobrecargas aceptarán ahora un parámetro de entrada adicional, que será un 
VectorType que representa la posición del pivote que se usará para aplicar 
las transformaciones. De este modo, el método Transform no tiene más que 
llamar a TransformFromPivot, pasando en el nuevo parámetro a 
VectorType::ZeroVector. Puede que en TransformFromPivot sea bueno colocar una 
comprobación de si el vector es nulo o no (IsZero), con el fin de optimizar 
rendimiento.

Original issue reported on code.google.com by Lince3D@gmail.com on 9 Apr 2011 at 8:29

[GoogleCodeExporter]

Status: Started, Owner: anderson_jag

Original comment by anderson...@gmail.com on 27 May 2011 at 2:06

• Changed state: Started

[GoogleCodeExporter]

Quedaba sin poner --> Estado: ReadyForReview

Original comment by anderson...@gmail.com on 21 Jul 2011 at 1:57

• Changed state: ReadyForReview

[GoogleCodeExporter]

Vuelo a dejarla en Started, para mandar otro parche.

Original comment by anderson...@gmail.com on 21 Jul 2011 at 2:10

• Changed state: Started

[GoogleCodeExporter]

Enviado nuevo parche.

Status: ReadyForReview.

Original comment by anderson...@gmail.com on 23 Jul 2011 at 9:12

• Changed state: ReadyForReview

[GoogleCodeExporter]

Añadidos los cambios especificados en la tarea.

Corregidos algunos nombres de parámetros en QLineSegment2D y QLineSegment3D

Añadidos en QLineSegment2D y QLineSegment3D el método Intersection que acepta 
un orb, y que llamará explícitamente al método Intersection que acepta un 
orb de QLineSegment.

Estado: ReadyForReview

Original comment by anderson...@gmail.com on 27 Jul 2011 at 7:11

[GoogleCodeExporter]

Resultados de la revisión:

-QLineSegment2D - Documentación de Transform que recibe un ángulo y tiene 
parámetro de salida: Returns no especifica lo que produce la función, sino lo 
que devuelve como valor de retorno.

-QLineSegment2D - Transform que recibe un ángulo y tiene parámetro de salida: 
¿Por qué ya no tiene el modificador const al final de la firma?

-QLineSegment2D - TransformFromPivot que recibe un ángulo: Mantén el orden de 
parámetros de los métodos ya definidos: rotación, pivote, output.

-QLineSegment2D - Documentación de TransformFromPivot que recibe un ángulo: 
Creo que esta frase sería mejor así "The pivot point around the rotation will 
be accomplished." --> "The pivot point which the rotation will be accomplished 
around.".

-QLineSegment3D - Constructor por defecto: Llamar al constructor de la clase 
base no es necesario.

-QLineSegment3D - Documentación de TransformFromPivot que recibe un 
cuaternión: Creo que esta frase sería mejor así "The pivot point around the 
rotation will be accomplished." --> "The pivot point which the rotation will be 
accomplished around.".

-QLineSegment3D - TransformFromPivot que recibe un cuaternión: Mantén el 
orden de parámetros de los métodos ya definidos: rotación, pivote, output.

-QLineSegment2D.cpp: Los puntos y coma detrás de las llaves de cierre de cada 
función sobran.

Original comment by Lince3D@gmail.com on 28 Jul 2011 at 11:25

• Changed state: Started

[GoogleCodeExporter]

Resueltas todas las indicaciones del anterior comentario.

Estado: ReadyForReview

Original comment by anderson...@gmail.com on 29 Jul 2011 at 5:55

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revision: Correcta.

Original comment by Lince3D@gmail.com on 30 Jul 2011 at 1:30

• Changed state: Done