From 337cfa220bbd7ed4f1b87990d1f52ad9c0f4a67b Mon Sep 17 00:00:00 2001
From: gdd <gdd>
Date: Wed, 26 Oct 2011 10:05:59 +0000
Subject: [PATCH] rnc: I made a mistake. It should be OK now

---
 src/GEOMImpl/GEOMImpl_BoxDriver.cxx | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/GEOMImpl/GEOMImpl_BoxDriver.cxx b/src/GEOMImpl/GEOMImpl_BoxDriver.cxx
index a3145b83c..01dd83897 100644
--- a/src/GEOMImpl/GEOMImpl_BoxDriver.cxx
+++ b/src/GEOMImpl/GEOMImpl_BoxDriver.cxx
@@ -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;
       }
-- 
2.39.2