From 13786aab83c711f4ab6776b11d30104efceb6dca Mon Sep 17 00:00:00 2001 From: gka Date: Thu, 18 Oct 2018 19:50:35 +0300 Subject: [PATCH] 0030250: Obtained shape is invalid due to presence of the loop containing two separate edges lying on the seam of the closed surface Holes with null area is excluded from result in the ShapeFix_Face::FixMissingSeam in order to avoid cut by seam of the surface --- src/ShapeFix/ShapeFix_Face.cxx | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/src/ShapeFix/ShapeFix_Face.cxx b/src/ShapeFix/ShapeFix_Face.cxx index e5843ab6fe..e793f08e1c 100644 --- a/src/ShapeFix/ShapeFix_Face.cxx +++ b/src/ShapeFix/ShapeFix_Face.cxx @@ -505,13 +505,6 @@ Standard_Boolean ShapeFix_Face::Perform() if ( NeedFix(myFixPeriodicDegenerated) ) this->FixPeriodicDegenerated(); - - if ( NeedFix ( myFixSmallAreaWireMode ) ) - { - if (FixSmallAreaWire(Standard_False)) - myStatus |= ShapeExtend::EncodeStatus(ShapeExtend_DONE4); - } - myResult = myFace; TopoDS_Shape savShape = myFace; //gka BUG 6555 // fix missing seam @@ -1715,6 +1708,11 @@ Standard_Boolean ShapeFix_Face::FixMissingSeam() else if ( wire.IsSame ( w2 ) ) wire = w21; else { + Handle(ShapeAnalysis_Wire) anAnalyzer = new ShapeAnalysis_Wire(wire, myFace, Precision()); + //To avoid addition of holes with null area (OCCT issue 0030250) + if (anAnalyzer->CheckSmallArea(wire)) + continue; + // other wires (not boundary) are considered as holes; make sure to have them oriented accordingly TopoDS_Shape curface = tmpF.EmptyCopied(); B.Add(curface,wire); -- 2.39.5