From: vsr <vsr@opencascade.com>
Date: Thu, 12 Oct 2017 14:06:06 +0000 (+0300)
Subject: Revert commit a1bc463e for issue 0023451 as it caused regressions
X-Git-Tag: V8_4_0rc1~1
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=c24f6c6618b310e9950b1d866aff81a0ddfd82b3;p=modules%2Fgeom.git

Revert commit a1bc463e for issue 0023451 as it caused regressions
---

diff --git a/src/BlockFix/BlockFix_UnionFaces.cxx b/src/BlockFix/BlockFix_UnionFaces.cxx
index b3d92b18f..35e9ff26e 100644
--- a/src/BlockFix/BlockFix_UnionFaces.cxx
+++ b/src/BlockFix/BlockFix_UnionFaces.cxx
@@ -244,9 +244,6 @@ static Standard_Boolean IsEdgeValidToMerge(const TopoDS_Edge &theEdge,
   Standard_Boolean isValid = Standard_True;
 
   if (BRep_Tool::IsClosed(theEdge, theFace)) {
-    // Mantis issue 0023451, now code corresponds to the comment to this method
-    isValid = Standard_False;
-
     // This is a seam edge. Check if there are another seam edges on the face.
     TopExp_Explorer anExp(theFace, TopAbs_EDGE);
 
@@ -262,9 +259,7 @@ static Standard_Boolean IsEdgeValidToMerge(const TopoDS_Edge &theEdge,
       TopoDS_Edge anEdge = TopoDS::Edge(aShEdge);
 
       if (BRep_Tool::IsClosed(anEdge, theFace)) {
-        // Mantis issue 0023451, now code corresponds to the comment to this method
-        //isValid = Standard_False;
-        isValid = Standard_True;
+        isValid = Standard_False;
         break;
       }
     }