Closed GoogleCodeExporter closed 9 years ago
Status: Started, Owner: anderson_jag
Original comment by anderson...@gmail.com
on 3 May 2011 at 1:10
Status: ReadyForReview.
Original comment by anderson...@gmail.com
on 7 May 2011 at 11:24
-Añadido operator= y método IntersectionPoint.
-Añadido método protected GetClosestPoints, para simplificar MinDistance
(QBaseLineSegment>VectorType>) e IntersectionPoint, los cuales poseían un
código practicamente idéntico.
-Añadida constante LineZero.
-Cambiados nombres vBaseLineSegment por segmt.
-Añadidas etiquetas <remarks> a Scale, ScaleFromA y SaleFromB y puesta
información remark que antes estaba en <summary>.
Original comment by anderson...@gmail.com
on 7 May 2011 at 11:28
Resultado de la revisión:
-IntersectionPoint: No está bien especificado en la documentación pero lo
haremos así, debería devolver un EQIntersections, y devolver infinito si las
líneas son coincidentes, igual que en IntersectionPoint de QPlane.
-GetClosestPoints: ¿Por qué usas una variable static? Su inicialización debe
ir en la misma línea. En las sentencias QE_ASSERT, se debe comprobar si los
valores son cero con != QFloat::_0.
Original comment by Lince3D@gmail.com
on 8 May 2011 at 12:04
Realizados los siguientes cambios:
-Ampliada la funcionalidad de IntersectionPoint para que devuelva un
EQIntersections dependiendo de la situación de los vectores (se ortan en un
punto, no se cortan, son paralelos, total o parcialmente coincidentes, etc.)
-Quitada la condición static de la variable bBothDegeneratedIntoPoints dentro
de GetClosestPoints.
Original comment by anderson...@gmail.com
on 12 May 2011 at 12:07
Sentencias QE_ASSERT: comprobados si los valores son cero con != QFloat::_0
Original comment by anderson...@gmail.com
on 12 May 2011 at 12:10
Resultado de la revisión: Correcta.
-Documentación de IntersectionPoint: "intersecion" = "intersection".
-IntersectionPoint: Pon un TODO en las conversiones a grados y radianes, ya que
en el futuro no serán necesarias.
-En IntersectionPoint: "QE_ASSERT( fSqrLengthProd != QFloat::_0 ); // Just for
safety, it cannot be happen." ¿Qué se pretende prevenir? "cannot be happen" =
"cannot happen".
Original comment by Lince3D@gmail.com
on 23 May 2011 at 8:05
Original comment by jwl...@gmail.com
on 18 Dec 2011 at 9:34
Original issue reported on code.google.com by
Lince3D@gmail.com
on 25 Apr 2011 at 9:08