Salome HOME
rnc: I made a mistake. It should be OK now
authorgdd <gdd>
Wed, 26 Oct 2011 10:05:59 +0000 (10:05 +0000)
committergdd <gdd>
Wed, 26 Oct 2011 10:05:59 +0000 (10:05 +0000)
src/GEOMImpl/GEOMImpl_BoxDriver.cxx

index a3145b83c590771bbf95f3beb98959ed969af51f..01dd83897e751515779a0d8142372a12694575e0 100644 (file)
@@ -39,6 +39,8 @@
 
 #include <Precision.hxx>
 
+#include<cmath>
+
 //=======================================================================
 //function : GetID
 //purpose  :
@@ -91,9 +93,9 @@ Standard_Integer GEOMImpl_BoxDriver::Execute(TFunction_Logbook& log) const
       gp_Pnt P1 = BRep_Tool::Pnt(TopoDS::Vertex(aShape1));
       gp_Pnt P2 = BRep_Tool::Pnt(TopoDS::Vertex(aShape2));
 
-      if (P1.X() - P2.X() < Precision::Confusion() || 
-          P1.Y() - P2.Y() < Precision::Confusion() || 
-          P1.Z() - P2.Z() < Precision::Confusion() ) {
+      if (abs(P1.X() - P2.X()) < Precision::Confusion() || 
+          abs(P1.Y() - P2.Y()) < Precision::Confusion() || 
+          abs(P1.Z() - P2.Z()) < Precision::Confusion() ) {
         StdFail_NotDone::Raise("Box can not be created, the points belong both to one of the OXY, OYZ or OZX planes");
         return 0;
       }