0024818: CLang warnings -Wlogical-not-parentheses

CLang warnings -Wlogical-not-parentheses were fixed.

diff --git a/src/AIS/AIS_Dimension.cxx b/src/AIS/AIS_Dimension.cxx
index 52c735679bf36c77d6a7b72844e277d3c423daab..d8da604057bfd378fc47501f7a74ed4f52c502a4 100755 (executable)
--- a/src/AIS/AIS_Dimension.cxx
+++ b/src/AIS/AIS_Dimension.cxx
@@ -1145,7 +1145,7 @@ Standard_Boolean AIS_Dimension::InitCircularDimension (const TopoDS_Shape& theSh
     }
 
     BRepAdaptor_Curve anAdaptedCurve (anEdge);
-    if (!anAdaptedCurve.GetType() == GeomAbs_Circle)
+    if (anAdaptedCurve.GetType() != GeomAbs_Circle)
     {
       return Standard_False;
     }

diff --git a/src/AIS/AIS_MultipleConnectedInteractive.cxx b/src/AIS/AIS_MultipleConnectedInteractive.cxx
index 0b195ffc46dae9fa36399696837cafa09dcbc0c2..fba2ca1522feb7adad6d2f9b5a2bbf4f88636604 100644 (file)
--- a/src/AIS/AIS_MultipleConnectedInteractive.cxx
+++ b/src/AIS/AIS_MultipleConnectedInteractive.cxx
@@ -89,7 +89,7 @@ void AIS_MultipleConnectedInteractive::Connect(const Handle(AIS_InteractiveObjec
 //=======================================================================
 Standard_Boolean AIS_MultipleConnectedInteractive::HasConnection() const 
 {
-  return (!myReferences.Length()==0);
+  return (myReferences.Length() != 0);
 }
 
 //=======================================================================

diff --git a/src/BRepFill/BRepFill_MultiLine.cxx b/src/BRepFill/BRepFill_MultiLine.cxx
index bf9356e81a6570a011bf35a4b38d79261f789bdb..2da32ebd75497f3e66fc90909c672ec218b55c7a 100644 (file)
--- a/src/BRepFill/BRepFill_MultiLine.cxx
+++ b/src/BRepFill/BRepFill_MultiLine.cxx
@@ -602,7 +602,7 @@ static gp_Pnt2d ValueOnFace(const Standard_Real        U,
 
   Standard_Real UU =0., Dist = Precision::Infinite(), D1, D2;
 
-  if ( !Ext.NbPoints() == 0 ) {    
+  if ( Ext.NbPoints() != 0 ) {
     UU = Ext.LowerDistanceParameter();
     Dist = Ext.LowerDistance();
   }

diff --git a/src/Bisector/Bisector_BisecCC.cxx b/src/Bisector/Bisector_BisecCC.cxx
index 2ef3b1af042c14693089ea4f755825bf281f6978..ce9c28b3fcb3c2ceec4444e58fe723573505bf99 100644 (file)
--- a/src/Bisector/Bisector_BisecCC.cxx
+++ b/src/Bisector/Bisector_BisecCC.cxx
@@ -164,7 +164,7 @@ void  Bisector_BisecCC::Perform(const Handle(Geom2d_Curve)& Cu1,
     }
   }
 
-  if ( !myPolygon.Length() == 0) {
+  if ( myPolygon.Length() != 0 ) {
     SupLastParameter();
     //----------------------------------------------
     // Construction of the polygon of the bissectrice.

diff --git a/src/HLRTest/HLRTest_DrawableEdgeTool.cxx b/src/HLRTest/HLRTest_DrawableEdgeTool.cxx
index 4b515bb4ffe28663daa92746788e1c113c2ae0a8..8b1afbd049cff654583bf6b2d79913fb168e4153 100644 (file)
--- a/src/HLRTest/HLRTest_DrawableEdgeTool.cxx
+++ b/src/HLRTest/HLRTest_DrawableEdgeTool.cxx
@@ -188,7 +188,7 @@ HLRTest_DrawableEdgeTool::DrawEdge (Draw_Display& D,
           It.NextVisible()) {
        It.Visible(sta,tolsta,end,tolend);
        D.MoveTo(ec.Value3D(sta));
-       if (!ec.GetType() == GeomAbs_Line) {
+       if (ec.GetType() != GeomAbs_Line) {
          Standard_Integer nbPnt = 100;
          Standard_Real step = (end-sta)/(nbPnt+1);
          
@@ -221,7 +221,7 @@ HLRTest_DrawableEdgeTool::DrawEdge (Draw_Display& D,
           It.NextHidden()) {
        It.Hidden(sta,tolsta,end,tolend);
        D.MoveTo(ec.Value3D(sta));
-       if (!ec.GetType() == GeomAbs_Line) {
+       if (ec.GetType() != GeomAbs_Line) {
          Standard_Integer nbPnt = 100;
          Standard_Real step = (end-sta)/(nbPnt+1);
          

diff --git a/src/IGESDimen/IGESDimen_NewGeneralNote.cxx b/src/IGESDimen/IGESDimen_NewGeneralNote.cxx
index ecdf16bfbcbe046e8074c347775b9594823b5938..90f7268c59ebe7b0694479781984dd6468980187 100644 (file)
--- a/src/IGESDimen/IGESDimen_NewGeneralNote.cxx
+++ b/src/IGESDimen/IGESDimen_NewGeneralNote.cxx
@@ -277,7 +277,7 @@ IGESDimen_NewGeneralNote::IGESDimen_NewGeneralNote ()    {  }
     Standard_Boolean  IGESDimen_NewGeneralNote::IsMirrored
   (const Standard_Integer Index) const 
 {
-  return (! theMirrorFlags->Value(Index) == 0);
+  return (theMirrorFlags->Value(Index) != 0);
 }
 
     Standard_Integer  IGESDimen_NewGeneralNote::RotateFlag

diff --git a/src/IGESToBRep/IGESToBRep_TopoCurve.cxx b/src/IGESToBRep/IGESToBRep_TopoCurve.cxx
index a5b7e53f27a245578d7983cc3017186b52211100..0930dac2a7277a0c5794ed53162c8126585478c4 100644 (file)
--- a/src/IGESToBRep/IGESToBRep_TopoCurve.cxx
+++ b/src/IGESToBRep/IGESToBRep_TopoCurve.cxx
@@ -589,7 +589,7 @@ TopoDS_Shape  IGESToBRep_TopoCurve::TransferOffsetCurve
     //AddFail(start, "OffsetCurve Transfer Error : Null IGESEntity");
     return  res;
   }
-  if (!start->OffsetType()==1) {
+  if (start->OffsetType() != 1) {
     Message_Msg Msg1100("IGES_1100"); //"Offset distance flag different from 1 not treated"
     SendFail(start,Msg1100);
     //AddFail(start, "Offset distance flag different from 1 not treated");
@@ -609,7 +609,7 @@ TopoDS_Shape  IGESToBRep_TopoCurve::TransferOffsetCurve
 
   while (BaseCrv->IsKind(STANDARD_TYPE(IGESGeom_OffsetCurve))) {
     DeclareAndCast(IGESGeom_OffsetCurve, OffCrv, BaseCrv);
-    if (!OffCrv->OffsetType()==1) {
+    if (OffCrv->OffsetType() != 1) {
        Message_Msg Msg1100("IGES_1100"); //"Offset distance flag different from 1 not treated"
        SendFail(start,Msg1100);
        return  res;
@@ -746,7 +746,7 @@ TopoDS_Shape  IGESToBRep_TopoCurve::Transfer2dOffsetCurve
     return  res;
   }
 
-  if (!start->OffsetType()==1) {
+  if (start->OffsetType() != 1) {
     Message_Msg Msg1100("IGES_1100"); //"Offset distance flag different from 1 not treated"
     SendFail(start,Msg1100);
   }

diff --git a/src/LocOpe/LocOpe_BuildShape.cxx b/src/LocOpe/LocOpe_BuildShape.cxx
index df5882d503985b0078f5d5ed733045ec8c351af5..5fa3133b622317adf05d0bdd5c568930b547460d 100644 (file)
--- a/src/LocOpe/LocOpe_BuildShape.cxx
+++ b/src/LocOpe/LocOpe_BuildShape.cxx
@@ -261,7 +261,7 @@ void LocOpe_BuildShape::Perform(const TopTools_ListOfShape& L)
       }
       imbSh.Clear();
     }
-  } while (!imbSh.Extent() == 0);
+  } while (imbSh.Extent() != 0);
 
   Standard_Integer nbsol = lsolid.Extent();
   Standard_Integer nbshl = lshell.Extent();

diff --git a/src/MAT/MAT_Arc.cxx b/src/MAT/MAT_Arc.cxx
index fee90f2ba8dc052d681aaa7d295c1ee172ffa521..c914b4b4e32e32d559a236c57c8843371e722676 100644 (file)
--- a/src/MAT/MAT_Arc.cxx
+++ b/src/MAT/MAT_Arc.cxx
@@ -115,15 +115,15 @@ Standard_Boolean MAT_Arc::HasNeighbour(const Handle(MAT_Node)& aNode,
 {
   if (aSide == MAT_Left) {
 //    if (aNode == FirstNode())  return (!firstArcLeft  == NULL);
-    if (aNode == FirstNode())  return (!firstArcLeft  == 0);
+    if (aNode == FirstNode())  return (firstArcLeft != 0);
 //    if (aNode == SecondNode()) return (!secondArcLeft == NULL);
-    if (aNode == SecondNode()) return (!secondArcLeft == 0);
+    if (aNode == SecondNode()) return (secondArcLeft != 0);
   }
   else {
 //    if (aNode == FirstNode())  return (!firstArcRight  == NULL);
-    if (aNode == FirstNode())  return (!firstArcRight  == 0);
+    if (aNode == FirstNode())  return (firstArcRight != 0);
 //    if (aNode == SecondNode()) return (!secondArcRight == NULL);
-    if (aNode == SecondNode()) return (!secondArcRight == 0);
+    if (aNode == SecondNode()) return (secondArcRight != 0);
   }
   Standard_DomainError::Raise("MAT_Arc::HasNeighbour");
   return Standard_False;

diff --git a/src/StlTransfer/StlTransfer.cxx b/src/StlTransfer/StlTransfer.cxx
index 163e566fe54de83c8ed62d745b200dbfde540eab..bd1da2c240e14c0c999f70aac614b0fbbaf6e529 100644 (file)
--- a/src/StlTransfer/StlTransfer.cxx
+++ b/src/StlTransfer/StlTransfer.cxx
@@ -61,7 +61,7 @@ static void Normal(const TopoDS_Face&  aFace,
     CSLib_NormalStatus NStat;
     S.Initialize(aFace, Standard_False);
     const TColgp_Array1OfPnt2d& UVNodes = T->UVNodes();
-    if (!S.GetType() == GeomAbs_Plane) {
+    if (S.GetType() != GeomAbs_Plane) {
       for (i = UVNodes.Lower(); i <= UVNodes.Upper(); i++) {
        U = UVNodes(i).X();
        V = UVNodes(i).Y();

diff --git a/src/TNaming/TNaming_NamedShape.cxx b/src/TNaming/TNaming_NamedShape.cxx
index 9fe70fc98be8ea2edabb2960f9651f3e92a84371..21b634f556851f7ed4df77cd564ff401cedb3337 100644 (file)
--- a/src/TNaming/TNaming_NamedShape.cxx
+++ b/src/TNaming/TNaming_NamedShape.cxx
@@ -557,7 +557,7 @@ void TNaming_NamedShape::References(const Handle(TDF_DataSet)& aDataSet) const
 void TNaming_NamedShape::Add(TNaming_Node*& pdn )
 { 
   pdn->myAtt             = this;
-  if (!myNode == 0L){
+  if (myNode != 0L){
     pdn->nextSameAttribute = myNode;
   }
   myNode = pdn;