Add the splitting surfaces for separating the cones and other surfaces

shhzhoullxie · Mar 10, 2017 · e365fde · e365fde
1 parent 41a6055
commit e365fde
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/McCadDecompose/McCadSplitCylinders.cxx b/src/McCadDecompose/McCadSplitCylinders.cxx
@@ -120,7 +120,7 @@ Standard_Boolean McCadSplitCylinders::HasComCurvEdge(McCadBndSurfCylinder *&pSur
         for(unsigned j = 0; j < pSurfB->GetEdgeList().size(); j++)
         {
             McCadEdge *pEdgeB = pSurfB->GetEdgeList().at(j);
-            if(pEdgeA->GetType() != Ellipse)
+            if(pEdgeB->GetType() != Ellipse)
             {
                 continue;
             }
@@ -229,8 +229,8 @@ McCadAstSurfPlane * McCadSplitCylinders::GenSurfThroughTwoEdges( McCadEdgeLine *
 * @param TopoDS_Edge &edge
 * @return McCadAstSurfPlane
 *
-* @date 08/04/2016
-* @modify 08/04/2015
+* @date 08/04/2015
+* @modify 08/04/2016
 * @author  Lei Lu
 ******************************************************************************/
 McCadAstSurfPlane * McCadSplitCylinders::GenSurfThroughCurves(McCadEdge *& pEdge)

diff --git a/src/McCadTool/McCadGeomTool.cxx b/src/McCadTool/McCadGeomTool.cxx
@@ -1024,7 +1024,7 @@ Standard_Boolean McCadGeomTool::IsSameCircleEdge(const TopoDS_Edge &edgeA,
     Standard_Real fMidB = (fStartB+fEndB)/2.0;
     theCurveA->D0(fMidB, pntMidB);
 
-    // If the middle points are not same, the two ellipses are not same
+    // If the middle points are not same, the two circles are not same
     if(!pntMidA.IsEqual(pntMidB,1.0e-5))
     {
         return Standard_False;
@@ -1033,7 +1033,7 @@ Standard_Boolean McCadGeomTool::IsSameCircleEdge(const TopoDS_Edge &edgeA,
     gp_Circ circA = gp_CurveA.Circle();
     gp_Circ circB = gp_CurveB.Circle();
 
-    // Major and Minor radius are same
+    // The radius and locations are same
     if( Abs(circA.Radius() - circB.Radius()) <= dis
             && circA.Location().IsEqual(circB.Location(),1.0e-5))
     {