]> OCCT Git - occt-copy.git/commitdiff
Fixed exception when trying to calculate line and edge normal
authorvkuzin <vitaly.kuzin@opencascade.com>
Fri, 13 Aug 2021 11:43:22 +0000 (14:43 +0300)
committervkuzin <vitaly.kuzin@opencascade.com>
Fri, 13 Aug 2021 11:43:22 +0000 (14:43 +0300)
src/HLRBRep/HLRBRep_Data.cxx

index 72b0fdf3167dd9f33fc1287b1b1dc9ca1591bf28..a69882877be3ab2e4ded540161a89acdcd595791 100644 (file)
@@ -36,6 +36,7 @@
 #include <IntCurveSurface_TransitionOnCurve.hxx>
 #include <IntRes2d_IntersectionPoint.hxx>
 #include <IntRes2d_IntersectionSegment.hxx>
+#include <LProp_NotDefined.hxx>
 #include <Precision.hxx>
 #include <Standard_Type.hxx>
 #include <StdFail_UndefinedDerivative.hxx>
@@ -1545,41 +1546,44 @@ Standard_Boolean HLRBRep_Data::OrientOutLine (const Standard_Integer I, HLRBRep_
          V.Transform(TI);
          Standard_Real curv = HLRBRep_EdgeFaceTool::CurvatureValue
            (iFaceGeom,pu,pv,V);
-         gp_Vec Nm = mySLProps.Normal();
-         if (curv == 0) {
+         gp_Vec Nm;
+         if (mySLProps.IsNormalDefined()) {
+           Nm = mySLProps.Normal();
+           if (curv == 0) {
 #ifdef OCCT_DEBUG
-           std::cout << "HLRBRep_Data::OrientOutLine " << I;
-           std::cout << " Edge " << myFE << " : ";
-           std::cout << "CurvatureValue == 0." << std::endl;
+             std::cout << "HLRBRep_Data::OrientOutLine " << I;
+             std::cout << " Edge " << myFE << " : ";
+             std::cout << "CurvatureValue == 0." << std::endl;
 #endif
-         }
-         if (curv > 0)
-           Nm.Reverse();
-         Tg.Transform(T);
-         Pt.Transform(T);
-         Nm.Transform(T);
-         Nm.Cross(Tg);
-         if (Tg.Magnitude() < gp::Resolution()) {
+           }
+           if (curv > 0)
+             Nm.Reverse();
+           Tg.Transform(T);
+           Pt.Transform(T);
+           Nm.Transform(T);
+           Nm.Cross(Tg);
+           if (Tg.Magnitude() < gp::Resolution()) {
 #ifdef OCCT_DEBUG
-           std::cout << "HLRBRep_Data::OrientOutLine " << I;
-           std::cout << " Edge " << myFE << " : ";
-           std::cout << "Tg.Magnitude() == 0." << std::endl;
+             std::cout << "HLRBRep_Data::OrientOutLine " << I;
+             std::cout << " Edge " << myFE << " : ";
+             std::cout << "Tg.Magnitude() == 0." << std::endl;
 #endif  
-         }
-         if (myProj.Perspective())
-           r = Nm.Z() * myProj.Focus() - 
-             ( Nm.X() * Pt.X() + Nm.Y() * Pt.Y() + Nm.Z() * Pt.Z() );
-         else
-           r = Nm.Z();
-         myFEOri = (r > 0) ? TopAbs_FORWARD : TopAbs_REVERSED;
-         if (!FD.Cut() && FD.Closed() && FirstInversion) {
-           if ((eb1->Orientation(ie1) == myFEOri) != 
+           }
+           if (myProj.Perspective())
+             r = Nm.Z() * myProj.Focus() -
+             (Nm.X() * Pt.X() + Nm.Y() * Pt.Y() + Nm.Z() * Pt.Z());
+           else
+             r = Nm.Z();
+           myFEOri = (r > 0) ? TopAbs_FORWARD : TopAbs_REVERSED;
+           if (!FD.Cut() && FD.Closed() && FirstInversion) {
+             if ((eb1->Orientation(ie1) == myFEOri) !=
                (FD.Orientation() == TopAbs_FORWARD)) {
-             FirstInversion = Standard_False;
-             inverted = Standard_True;
+               FirstInversion = Standard_False;
+               inverted = Standard_True;
+             }
            }
+           eb1->Orientation(ie1, myFEOri);
          }
-         eb1->Orientation(ie1,myFEOri);
        }
        else {
 #ifdef OCCT_DEBUG
@@ -1627,19 +1631,22 @@ void HLRBRep_Data::OrientOthEdge (const Standard_Integer I,
        if (HLRBRep_EdgeFaceTool::UVPoint(p,myFEGeom,iFaceGeom,pu,pv)) {
          gp_Pnt Pt = EC.Value3D(p);
          mySLProps.SetParameters(pu,pv);
-         gp_Vec Nm = mySLProps.Normal();
-         Pt.Transform(T);
-         Nm.Transform(T);
-         if (myProj.Perspective()) {
-           r = Nm.Z() * myProj.Focus() - 
-             ( Nm.X() * Pt.X() + Nm.Y() * Pt.Y() + Nm.Z() * Pt.Z() );
-         }
-         else {
-           r = Nm.Z();
-         }
-         if (r < 0) {
-           myFEOri = TopAbs::Reverse(myFEOri);
-           eb1->Orientation(ie1,myFEOri);
+         gp_Vec Nm;
+         if (mySLProps.IsNormalDefined()) {
+           Nm = mySLProps.Normal();
+           Pt.Transform(T);
+           Nm.Transform(T);
+           if (myProj.Perspective()) {
+             r = Nm.Z() * myProj.Focus() -
+               (Nm.X() * Pt.X() + Nm.Y() * Pt.Y() + Nm.Z() * Pt.Z());
+           }
+           else {
+             r = Nm.Z();
+           }
+           if (r < 0) {
+             myFEOri = TopAbs::Reverse(myFEOri);
+             eb1->Orientation(ie1, myFEOri);
+           }
          }
        }
 #ifdef OCCT_DEBUG