0024893: CLang warnings -Wlogical-not-parentheses for gce_Done comparisons
authorabv <abv@opencascade.com>
Sat, 3 May 2014 04:58:18 +0000 (08:58 +0400)
committerbugmaster <bugmaster@opencascade.com>
Thu, 8 May 2014 07:57:59 +0000 (11:57 +0400)
Incorrect comparison in condition within RiseIf macro fixed

39 files changed:
src/GC/GC_MakeArcOfCircle.cxx
src/GC/GC_MakeArcOfEllipse.cxx
src/GC/GC_MakeArcOfHyperbola.cxx
src/GC/GC_MakeArcOfParabola.cxx
src/GC/GC_MakeCircle.cxx
src/GC/GC_MakeConicalSurface.cxx
src/GC/GC_MakeCylindricalSurface.cxx
src/GC/GC_MakeEllipse.cxx
src/GC/GC_MakeHyperbola.cxx
src/GC/GC_MakeLine.cxx
src/GC/GC_MakePlane.cxx
src/GC/GC_MakeSegment.cxx
src/GC/GC_MakeTrimmedCone.cxx
src/GC/GC_MakeTrimmedCylinder.cxx
src/GCE2d/GCE2d_MakeArcOfCircle.cxx
src/GCE2d/GCE2d_MakeArcOfEllipse.cxx
src/GCE2d/GCE2d_MakeArcOfHyperbola.cxx
src/GCE2d/GCE2d_MakeArcOfParabola.cxx
src/GCE2d/GCE2d_MakeCircle.cxx
src/GCE2d/GCE2d_MakeEllipse.cxx
src/GCE2d/GCE2d_MakeHyperbola.cxx
src/GCE2d/GCE2d_MakeLine.cxx
src/GCE2d/GCE2d_MakeParabola.cxx
src/GCE2d/GCE2d_MakeSegment.cxx
src/gce/gce_MakeCirc.cxx
src/gce/gce_MakeCirc2d.cxx
src/gce/gce_MakeCone.cxx
src/gce/gce_MakeCylinder.cxx
src/gce/gce_MakeDir.cxx
src/gce/gce_MakeDir2d.cxx
src/gce/gce_MakeElips.cxx
src/gce/gce_MakeElips2d.cxx
src/gce/gce_MakeHypr.cxx
src/gce/gce_MakeHypr2d.cxx
src/gce/gce_MakeLin.cxx
src/gce/gce_MakeLin2d.cxx
src/gce/gce_MakeParab.cxx
src/gce/gce_MakeParab2d.cxx
src/gce/gce_MakePln.cxx

index 9b88a58..41d22e2 100644 (file)
@@ -158,7 +158,7 @@ GC_MakeArcOfCircle::GC_MakeArcOfCircle(const gp_Circ& Circ   ,
 //=======================================================================
 const Handle(Geom_TrimmedCurve)& GC_MakeArcOfCircle::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheArc;
 }
 //=======================================================================
index c6b39d4..fc859d8 100644 (file)
@@ -54,7 +54,7 @@ GC_MakeArcOfEllipse::GC_MakeArcOfEllipse(const gp_Elips& Elips   ,
 
 const Handle(Geom_TrimmedCurve)& GC_MakeArcOfEllipse::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheArc;
 }
 
index ed95e3f..26bf645 100644 (file)
@@ -55,7 +55,7 @@ GC_MakeArcOfHyperbola::GC_MakeArcOfHyperbola(const gp_Hypr&        Hypr   ,
 
 const Handle(Geom_TrimmedCurve)& GC_MakeArcOfHyperbola::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheArc;
 }
 
index 69c5c31..acb1d00 100644 (file)
@@ -54,7 +54,7 @@ GC_MakeArcOfParabola::GC_MakeArcOfParabola(const gp_Parab& Parab ,
 
 const Handle(Geom_TrimmedCurve)& GC_MakeArcOfParabola::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheArc;
 }
 
index 3aa1e45..3caac67 100644 (file)
@@ -97,7 +97,7 @@ GC_MakeCircle::GC_MakeCircle(const gp_Ax1& Axis   ,
 
 const Handle(Geom_Circle)& GC_MakeCircle::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheCircle;
 }
 
index 357baca..aa8546c 100644 (file)
@@ -105,7 +105,7 @@ GC_MakeConicalSurface::GC_MakeConicalSurface(const gp_Pnt&       P1 ,
 
 const Handle(Geom_ConicalSurface)& GC_MakeConicalSurface::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheCone;
 }
 
index 39983c6..9d8a9a6 100644 (file)
@@ -98,7 +98,7 @@ GC_MakeCylindricalSurface::GC_MakeCylindricalSurface(const gp_Cylinder& Cyl ,
 const Handle(Geom_CylindricalSurface)& 
        GC_MakeCylindricalSurface::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheCylinder;
 }
 
index 637d2af..9e61d84 100644 (file)
@@ -48,7 +48,7 @@ GC_MakeEllipse::GC_MakeEllipse(const gp_Pnt& S1     ,
 
 const Handle(Geom_Ellipse)& GC_MakeEllipse::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheEllipse;
 }
 
index 5821767..d9cb199 100644 (file)
@@ -47,7 +47,7 @@ GC_MakeHyperbola::GC_MakeHyperbola(const gp_Pnt& S1     ,
 
 const Handle(Geom_Hyperbola)& GC_MakeHyperbola::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheHyperbola;
 }
 
index cfa48eb..617f291 100644 (file)
@@ -63,7 +63,7 @@ GC_MakeLine::GC_MakeLine(const gp_Lin& Lin   ,
 
 const Handle(Geom_Line)& GC_MakeLine::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheLine;
 }
 
index f98d339..88232f5 100644 (file)
@@ -119,7 +119,7 @@ GC_MakePlane::GC_MakePlane(const gp_Ax1& Axis ) {
 
 const Handle(Geom_Plane)& GC_MakePlane::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return ThePlane;
 }
 
index 13e6eac..35589b6 100644 (file)
@@ -61,7 +61,7 @@ GC_MakeSegment::GC_MakeSegment(const gp_Lin& Line  ,
 
 const Handle(Geom_TrimmedCurve)& GC_MakeSegment::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheSegment;
 }
 
index e708d6e..a4beefe 100644 (file)
@@ -73,7 +73,7 @@ GC_MakeTrimmedCone::GC_MakeTrimmedCone(const gp_Pnt&       P1 ,
 const Handle(Geom_RectangularTrimmedSurface)& GC_MakeTrimmedCone::
        Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheCone;
 }
 
index 269df86..2aac425 100644 (file)
@@ -82,7 +82,7 @@ GC_MakeTrimmedCylinder::GC_MakeTrimmedCylinder(const gp_Cylinder& , // Cyl,
 const Handle(Geom_RectangularTrimmedSurface)& GC_MakeTrimmedCylinder::
        Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheCyl;
 }
 
index 4f1da0b..0988646 100644 (file)
@@ -111,7 +111,7 @@ GCE2d_MakeArcOfCircle::GCE2d_MakeArcOfCircle(const gp_Circ2d&       Circ   ,
 
 const Handle(Geom2d_TrimmedCurve)& GCE2d_MakeArcOfCircle::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheArc;
 }
 
index 1bb9acf..07fbd2f 100644 (file)
@@ -54,7 +54,7 @@ GCE2d_MakeArcOfEllipse::GCE2d_MakeArcOfEllipse(const gp_Elips2d&      Elips  ,
 
 const Handle(Geom2d_TrimmedCurve)& GCE2d_MakeArcOfEllipse::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheArc;
 }
 
index 668185d..d981da8 100644 (file)
@@ -56,7 +56,7 @@ GCE2d_MakeArcOfHyperbola::
 
 const Handle(Geom2d_TrimmedCurve)& GCE2d_MakeArcOfHyperbola::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheArc;
 }
 
index 47c9048..613e6e2 100644 (file)
@@ -55,7 +55,7 @@ GCE2d_MakeArcOfParabola::
 
 const Handle(Geom2d_TrimmedCurve)& GCE2d_MakeArcOfParabola::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheArc;
 }
 
index 1a08db8..d8058e7 100644 (file)
@@ -100,7 +100,7 @@ GCE2d_MakeCircle::GCE2d_MakeCircle(const gp_Pnt2d&        Center ,
 
 const Handle(Geom2d_Circle)& GCE2d_MakeCircle::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheCircle;
 }
 
index 962b017..4800102 100644 (file)
@@ -59,7 +59,7 @@ GCE2d_MakeEllipse::GCE2d_MakeEllipse(const gp_Pnt2d& S1     ,
 
 const Handle(Geom2d_Ellipse)& GCE2d_MakeEllipse::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheEllipse;
 }
 
index 55a6ef2..714edf3 100644 (file)
@@ -59,7 +59,7 @@ GCE2d_MakeHyperbola::GCE2d_MakeHyperbola(const gp_Pnt2d& S1     ,
 
 const Handle(Geom2d_Hyperbola)& GCE2d_MakeHyperbola::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheHyperbola;
 }
 
index 043b8b9..a3122da 100644 (file)
@@ -73,7 +73,7 @@ GCE2d_MakeLine::GCE2d_MakeLine(const gp_Lin2d&     Lin  ,
 
 const Handle(Geom2d_Line)& GCE2d_MakeLine::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheLine;
 }
 
index 823459d..381ee1f 100644 (file)
@@ -74,7 +74,7 @@ GCE2d_MakeParabola::GCE2d_MakeParabola(const gp_Pnt2d& S1 ,
 
 const Handle(Geom2d_Parabola)& GCE2d_MakeParabola::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheParabola;
 }
 
index 2d4fc0c..4060178 100644 (file)
@@ -77,7 +77,7 @@ GCE2d_MakeSegment::GCE2d_MakeSegment(const gp_Lin2d&     Line  ,
 
 const Handle(Geom2d_TrimmedCurve)& GCE2d_MakeSegment::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheSegment;
 }
 
index 063b21f..ffd34e4 100644 (file)
@@ -338,7 +338,7 @@ gce_MakeCirc::gce_MakeCirc(const gp_Circ& Circ ,
 //=======================================================================
 const gp_Circ& gce_MakeCirc::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheCirc;
 }
 //=======================================================================
index 3c56740..a1bd93d 100644 (file)
@@ -202,7 +202,7 @@ gce_MakeCirc2d::gce_MakeCirc2d(const gp_Circ2d&    Circ  ,
 
 const gp_Circ2d& gce_MakeCirc2d::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheCirc2d;
 }
 
index 3d0bf9f..abc4fbe 100644 (file)
@@ -198,7 +198,7 @@ gce_MakeCone::gce_MakeCone(const gp_Pnt&       P1   ,
 
 const gp_Cone& gce_MakeCone::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheCone;
 }
 
index d010b6f..379c969 100644 (file)
@@ -126,7 +126,7 @@ gce_MakeCylinder::gce_MakeCylinder(const gp_Cylinder& Cyl ,
 
 const gp_Cylinder& gce_MakeCylinder::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheCylinder;
 }
 
index 607b5ef..8a36205 100644 (file)
@@ -63,7 +63,7 @@ gce_MakeDir::gce_MakeDir(const Standard_Real Xv,
 
 const gp_Dir& gce_MakeDir::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheDir;
 }
 
index 9236178..107b7d3 100644 (file)
@@ -62,7 +62,7 @@ gce_MakeDir2d::gce_MakeDir2d(const Standard_Real Xv,
 
 const gp_Dir2d& gce_MakeDir2d::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheDir2d;
 }
 
index c79f572..9916e21 100644 (file)
@@ -63,7 +63,7 @@ gce_MakeElips::gce_MakeElips(const gp_Pnt&   S1     ,
 
 const gp_Elips& gce_MakeElips::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheElips;
 }
 
index 90a193b..ebdeb58 100644 (file)
@@ -69,7 +69,7 @@ gce_MakeElips2d::gce_MakeElips2d(const gp_Ax22d&     A           ,
 
 const gp_Elips2d& gce_MakeElips2d::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheElips2d;
 }
 
index 75a8608..c353741 100644 (file)
@@ -55,7 +55,7 @@ gce_MakeHypr::gce_MakeHypr(const gp_Ax2&       A2          ,
 
 const gp_Hypr& gce_MakeHypr::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheHypr;
 }
 
index a89aa44..08595aa 100644 (file)
@@ -67,7 +67,7 @@ gce_MakeHypr2d::gce_MakeHypr2d(const gp_Ax22d&     A           ,
 
 const gp_Hypr2d& gce_MakeHypr2d::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheHypr2d;
 }
 
index 8c942da..59eabbd 100644 (file)
@@ -69,7 +69,7 @@ gce_MakeLin::gce_MakeLin(const gp_Lin& Lin ,
 
 const gp_Lin& gce_MakeLin::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheLin;
 }
 
index 6d002b1..892b034 100644 (file)
@@ -103,7 +103,7 @@ gce_MakeLin2d::gce_MakeLin2d(const gp_Lin2d&     Line,
 
 gp_Lin2d gce_MakeLin2d::Value() const
 { 
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheLin2d;
 }
 
index b1bf086..9c14815 100644 (file)
@@ -37,7 +37,7 @@ gce_MakeParab::gce_MakeParab(const gp_Ax1& D ,
 
 const gp_Parab& gce_MakeParab::Value () const
 {
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheParab;
 }
 
index 93d6e5f..61f586b 100644 (file)
@@ -75,7 +75,7 @@ gce_MakeParab2d::gce_MakeParab2d(const gp_Pnt2d&        S      ,
 
 const gp_Parab2d& gce_MakeParab2d::Value () const
 {
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return TheParab2d;
 }
 
index c002352..8aeadab 100644 (file)
@@ -124,7 +124,7 @@ gce_MakePln::gce_MakePln(const gp_Ax1& Axis )
 */
 const gp_Pln& gce_MakePln::Value () const
 {
-  StdFail_NotDone_Raise_if(!TheError == gce_Done,"");
+  StdFail_NotDone_Raise_if(TheError != gce_Done,"");
   return ThePln;
 }