0024818: CLang warnings -Wlogical-not-parentheses
authorski <ski@opencascade.com>
Wed, 16 Apr 2014 11:59:21 +0000 (15:59 +0400)
committerabv <abv@opencascade.com>
Mon, 21 Apr 2014 15:58:42 +0000 (19:58 +0400)
CLang warnings -Wlogical-not-parentheses were fixed.

src/AIS/AIS_Dimension.cxx
src/AIS/AIS_MultipleConnectedInteractive.cxx
src/BRepFill/BRepFill_MultiLine.cxx
src/Bisector/Bisector_BisecCC.cxx
src/HLRTest/HLRTest_DrawableEdgeTool.cxx
src/IGESDimen/IGESDimen_NewGeneralNote.cxx
src/IGESToBRep/IGESToBRep_TopoCurve.cxx
src/LocOpe/LocOpe_BuildShape.cxx
src/MAT/MAT_Arc.cxx
src/StlTransfer/StlTransfer.cxx
src/TNaming/TNaming_NamedShape.cxx

index 52c7356..d8da604 100755 (executable)
@@ -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;
     }
index 0b195ff..fba2ca1 100644 (file)
@@ -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);
 }
 
 //=======================================================================
index bf9356e..2da32eb 100644 (file)
@@ -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();
   }
index 2ef3b1a..ce9c28b 100644 (file)
@@ -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.
index 4b515bb..8b1afbd 100644 (file)
@@ -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);
          
index ecdf16b..90f7268 100644 (file)
@@ -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
index a5b7e53..0930dac 100644 (file)
@@ -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);
   }
index df5882d..5fa3133 100644 (file)
@@ -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();
index fee90f2..c914b4b 100644 (file)
@@ -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;
index 163e566..bd1da2c 100644 (file)
@@ -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();
index 9fe70fc..21b634f 100644 (file)
@@ -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;