From: gka <gka@opencascade.com>
Date: Thu, 18 Oct 2018 16:50:35 +0000 (+0300)
Subject: 0030250: Obtained shape is invalid due to presence of the loop containing two separat... 
X-Git-Url: http://git.dev.opencascade.org/gitweb/?a=commitdiff_plain;h=13786aab83c711f4ab6776b11d30104efceb6dca;p=occt-copy.git

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
---

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);