0023144: Suspicious if (3)
authoromy <omy@opencascade.com>
Thu, 15 Nov 2012 09:01:32 +0000 (13:01 +0400)
committeromy <omy@opencascade.com>
Thu, 15 Nov 2012 09:01:32 +0000 (13:01 +0400)
Fixed mistake in Standard_Boolean IntCoCo ( ... ) function of IntPatch_ImpImpIntersection_5.gxx file.
Fixed this condition because of the opposite parametrisation direction of the intersection line.

src/IntPatch/IntPatch_ImpImpIntersection_5.gxx

index 0aac0dc..2320d6c 100755 (executable)
@@ -109,9 +109,10 @@ Standard_Boolean IntCoCo(const IntSurf_Quadric& Quad1,
        //
        situC1=IntSurf_Inside;
        situC2=IntSurf_Outside;
-       if (aR1>aR1) {
-         situC1=IntSurf_Outside;
-         situC2=IntSurf_Inside;
+       if (aR1>aR2) {                                  // Intersection line parametrizes from Apex1 to Apex2,
+         situC1=IntSurf_Outside;               // So the distance betwee ptbid and aLAx1 is greater than the 
+         situC2=IntSurf_Inside;                // distance between ptbid and aLAx2 and in that case Cone2 
+                                                                       // is inside Cone 1
        }
       }
       // 1