From 3f8a20c6077723c1a4b79a6753eff2441da8e9e4 Mon Sep 17 00:00:00 2001
From: gdd <gdd>
Date: Thu, 19 Apr 2012 12:30:59 +0000
Subject: [PATCH] Fix a bug + Debug

---
 src/GEOMImpl/GEOMImpl_IAdvancedOperations.cxx | 26 ++++++++++++++++---
 1 file changed, 22 insertions(+), 4 deletions(-)

diff --git a/src/GEOMImpl/GEOMImpl_IAdvancedOperations.cxx b/src/GEOMImpl/GEOMImpl_IAdvancedOperations.cxx
index 810b6ceb8..cf7d7af71 100644
--- a/src/GEOMImpl/GEOMImpl_IAdvancedOperations.cxx
+++ b/src/GEOMImpl/GEOMImpl_IAdvancedOperations.cxx
@@ -689,12 +689,15 @@ bool GEOMImpl_IAdvancedOperations::MakePipeTShapePartition(Handle(GEOM_Object) t
       v->GetLastFunction()->SetDescription("");
       TopoDS_Vertex aVertex = TopoDS::Vertex(v->GetValue());
       gp_Pnt aP = BRep_Tool::Pnt(aVertex);
+//       std::cout << "Coords: " << aP.X() << ", " << aP.Y() << ", " << aP.Z() << std::endl;
       if (Abs(aP.X()) <= Precision::Confusion()) {
-        if (Abs(aP.Y()) - theR2 <= Precision::Confusion())
+        if (Abs(aP.Y()) - theR1 <= Precision::Confusion()) {
           vi1 = v;
+        }
       } else if (Abs(aP.Y()) <= Precision::Confusion()) {
-        if (Abs(aP.X()) - theR1 <= Precision::Confusion())
+        if (Abs(aP.X()) - theR1 <= Precision::Confusion()) {
           vi2 = v;
+        }
       }
     }
 
@@ -740,13 +743,18 @@ bool GEOMImpl_IAdvancedOperations::MakePipeTShapePartition(Handle(GEOM_Object) t
         v->GetLastFunction()->SetDescription("");
         TopoDS_Vertex aVertex = TopoDS::Vertex(v->GetValue());
         gp_Pnt aP = BRep_Tool::Pnt(aVertex);
+//         std::cout << "Coords: " << aP.X() << ", " << aP.Y() << ", " << aP.Z() << std::endl;
         if (Abs(aP.X()) <= Precision::Confusion()) {
-          if (Abs(aP.Y()) - theR2 > Precision::Confusion())
+          if (Abs(aP.Y()) - theR2 > Precision::Confusion()) {
             ve1 = v;
+          }
         } else if (Abs(aP.Y()) <= Precision::Confusion()) {
-          if (Abs(aP.X()) - theR2 > Precision::Confusion())
+          if (Abs(aP.X()) - theR2 > Precision::Confusion()) {
             ve2 = v;
+          }
         }
+        if ( !ve1.IsNull() && !ve2.IsNull())
+          break;
       }
       Handle(GEOM_Object) edge_e1, edge_e2;
       
@@ -782,6 +790,16 @@ bool GEOMImpl_IAdvancedOperations::MakePipeTShapePartition(Handle(GEOM_Object) t
         return false;
       }
       face_t->GetLastFunction()->SetDescription("");
+      
+      theShapes.push_back(theShape);
+      theShapes.push_back(vi1);
+      theShapes.push_back(vi2);
+      theShapes.push_back(ve1);
+      theShapes.push_back(ve2);
+      theShapes.push_back(edge_e1);
+      theShapes.push_back(edge_e2);
+      theShapes.push_back(wire_t);
+      theShapes.push_back(face_t);
     }
     else {
       Handle(GEOM_Object) P1, P2, P3, P4, P5, P6;
-- 
2.39.2