Coding - Apply Clang-Tidy automatic fixes

Flags:
aChecks="modernize-deprecated-headers,modernize-redundant-void-arg,modernize-use-bool-literals,modernize-use-equals-default,modernize-use-equals-delete,modernize-use-nullptr,modernize-use-override,performance-avoid-endl,performance-move-constructor-init,readability-braces-around-statements,readability-delete-null-pointer,readability-redundant-control-flow,readability-redundant-declaration,readability-redundant-function-ptr-dereference,readability-redundant-member-init,readability-redundant-string-init,readability-static-accessed-through-instance"

clang-tidy version: 22.1.3

Author: dpasukhi

src/ApplicationFramework/TKBin/BinDrivers/BinDrivers_DocumentRetrievalDriver.cxx

@@ -100,16 +100,22 @@ void BinDrivers_DocumentRetrievalDriver::EnableQuickPartReading(
   bool                                  theValue)
 {
   if (myDrivers.IsNull())
+  {
     myDrivers = AttributeDrivers(theMessageDriver);
+  }
   if (myDrivers.IsNull())
+  {
     return;
+  }
 
   occ::handle<BinMDF_ADriver> aDriver;
   myDrivers->GetDriver(STANDARD_TYPE(TNaming_NamedShape), aDriver);
   occ::handle<BinMNaming_NamedShapeDriver> aShapesDriver =
     occ::down_cast<BinMNaming_NamedShapeDriver>(aDriver);
   if (aShapesDriver.IsNull())
+  {
     throw Standard_NotImplemented("Internal Error - TNaming_NamedShape is not found!");
+  }
 
   aShapesDriver->EnableQuickPart(theValue);
 }

src/ApplicationFramework/TKBin/BinMDataXtd/BinMDataXtd_ConstraintDriver.cxx

@@ -53,12 +53,16 @@ bool BinMDataXtd_ConstraintDriver::Paste(const BinObjMgt_Persistent&       theSo
 
   // value
   if (!(theSource >> aNb))
+  {
     return false;
+  }
   if (aNb > 0)
   {
     occ::handle<TDataStd_Real> aTValue;
     if (theRelocTable.IsBound(aNb))
+    {
       aTValue = occ::down_cast<TDataStd_Real>(theRelocTable.Find(aNb));
+    }
     else
     {
       aTValue = new TDataStd_Real;
@@ -70,17 +74,23 @@ bool BinMDataXtd_ConstraintDriver::Paste(const BinObjMgt_Persistent&       theSo
   // geometries
   int NbGeom;
   if (!(theSource >> NbGeom))
+  {
     return false;
+  }
   int iG = 1;
   while (iG <= NbGeom)
   {
     if (!(theSource >> aNb))
+    {
       return false;
+    }
     if (aNb > 0)
     {
       occ::handle<TNaming_NamedShape> aG;
       if (theRelocTable.IsBound(aNb))
+      {
         aG = occ::down_cast<TNaming_NamedShape>(theRelocTable.Find(aNb));
+      }
       else
       {
         aG = new TNaming_NamedShape;
@@ -92,12 +102,16 @@ bool BinMDataXtd_ConstraintDriver::Paste(const BinObjMgt_Persistent&       theSo
 
   // plane
   if (!(theSource >> aNb))
+  {
     return false;
+  }
   if (aNb > 0)
   {
     occ::handle<TNaming_NamedShape> aTPlane;
     if (theRelocTable.IsBound(aNb))
+    {
       aTPlane = occ::down_cast<TNaming_NamedShape>(theRelocTable.Find(aNb));
+    }
     else
     {
       aTPlane = new TNaming_NamedShape;
@@ -109,13 +123,17 @@ bool BinMDataXtd_ConstraintDriver::Paste(const BinObjMgt_Persistent&       theSo
   // constraint type
   int aType;
   if (!(theSource >> aType))
+  {
     return false;
+  }
   aC->SetType((TDataXtd_ConstraintEnum)aType);
 
   // flags
   int flags;
   if (!(theSource >> flags))
+  {
     return false;
+  }
   aC->Verified((flags & 1) != 0);
   aC->Inverted((flags & 2) != 0);
   aC->Reversed((flags & 4) != 0);
@@ -137,9 +155,13 @@ void BinMDataXtd_ConstraintDriver::Paste(
   // value
   occ::handle<TDataStd_Real> aValue = aC->GetValue();
   if (!aValue.IsNull())
+  {
     aNb = theRelocTable.Add(aValue); // create and/or get index
+  }
   else
+  {
     aNb = -1;
+  }
   theTarget << aNb;
 
   // geometries
@@ -150,18 +172,26 @@ void BinMDataXtd_ConstraintDriver::Paste(
   {
     occ::handle<TNaming_NamedShape> aG = aC->GetGeometry(iG);
     if (!aG.IsNull())
+    {
       aNb = theRelocTable.Add(aG);
+    }
     else
+    {
       aNb = -1;
+    }
     theTarget << aNb;
   }
 
   // plane
   occ::handle<TNaming_NamedShape> aTPlane = aC->GetPlane();
   if (!aTPlane.IsNull())
+  {
     aNb = theRelocTable.Add(aTPlane);
+  }
   else
+  {
     aNb = -1;
+  }
   theTarget << aNb;
 
   // constraint type
@@ -170,10 +200,16 @@ void BinMDataXtd_ConstraintDriver::Paste(
   // flags
   int flags = 0;
   if (aC->Verified())
+  {
     flags |= 1;
+  }
   if (aC->Inverted())
+  {
     flags |= 2;
+  }
   if (aC->Reversed())
+  {
     flags |= 4;
+  }
   theTarget << flags;
 }

src/ApplicationFramework/TKBin/BinMDataXtd/BinMDataXtd_GeometryDriver.cxx

@@ -50,7 +50,9 @@ bool BinMDataXtd_GeometryDriver::Paste(const BinObjMgt_Persistent&       theSour
   int  aType;
   bool ok = theSource >> aType;
   if (ok)
+  {
     aT->SetType((TDataXtd_GeometryEnum)aType);
+  }
 
   return ok;
 }

src/ApplicationFramework/TKBin/BinMDataXtd/BinMDataXtd_PatternStdDriver.cxx

@@ -53,15 +53,21 @@ bool BinMDataXtd_PatternStdDriver::Paste(const BinObjMgt_Persistent&       theSo
   // signature
   int signature;
   if (!(theSource >> signature))
+  {
     return false;
+  }
   if (signature == 0)
+  {
     return true;
+  }
   aP->Signature(signature);
 
   // reversed flags
   int revFlags;
   if (!(theSource >> revFlags))
+  {
     return false;
+  }
   aP->Axis1Reversed((revFlags & 1) != 0);
   aP->Axis2Reversed((revFlags & 2) != 0);
 
@@ -71,9 +77,13 @@ bool BinMDataXtd_PatternStdDriver::Paste(const BinObjMgt_Persistent&       theSo
   if (signature == 5) // mirror
   {
     if (!(theSource >> aNb))
+    {
       return false;
+    }
     if (theRelocTable.IsBound(aNb))
+    {
       TNS = occ::down_cast<TNaming_NamedShape>(theRelocTable.Find(aNb));
+    }
     else
     {
       TNS = new TNaming_NamedShape;
@@ -88,9 +98,13 @@ bool BinMDataXtd_PatternStdDriver::Paste(const BinObjMgt_Persistent&       theSo
 
     // axis 1
     if (!(theSource >> aNb))
+    {
       return false;
+    }
     if (theRelocTable.IsBound(aNb))
+    {
       TNS = occ::down_cast<TNaming_NamedShape>(theRelocTable.Find(aNb));
+    }
     else
     {
       TNS = new TNaming_NamedShape;
@@ -100,9 +114,13 @@ bool BinMDataXtd_PatternStdDriver::Paste(const BinObjMgt_Persistent&       theSo
 
     // value 1
     if (!(theSource >> aNb))
+    {
       return false;
+    }
     if (theRelocTable.IsBound(aNb))
+    {
       TReal = occ::down_cast<TDataStd_Real>(theRelocTable.Find(aNb));
+    }
     else
     {
       TReal = new TDataStd_Real;
@@ -112,9 +130,13 @@ bool BinMDataXtd_PatternStdDriver::Paste(const BinObjMgt_Persistent&       theSo
 
     // number of instances 1
     if (!(theSource >> aNb))
+    {
       return false;
+    }
     if (theRelocTable.IsBound(aNb))
+    {
       TInt = occ::down_cast<TDataStd_Integer>(theRelocTable.Find(aNb));
+    }
     else
     {
       TInt = new TDataStd_Integer;
@@ -126,9 +148,13 @@ bool BinMDataXtd_PatternStdDriver::Paste(const BinObjMgt_Persistent&       theSo
     {
       // axis 2
       if (!(theSource >> aNb))
+      {
         return false;
+      }
       if (theRelocTable.IsBound(aNb))
+      {
         TNS = occ::down_cast<TNaming_NamedShape>(theRelocTable.Find(aNb));
+      }
       else
       {
         TNS = new TNaming_NamedShape;
@@ -138,9 +164,13 @@ bool BinMDataXtd_PatternStdDriver::Paste(const BinObjMgt_Persistent&       theSo
 
       // real value 2
       if (!(theSource >> aNb))
+      {
         return false;
+      }
       if (theRelocTable.IsBound(aNb))
+      {
         TReal = occ::down_cast<TDataStd_Real>(theRelocTable.Find(aNb));
+      }
       else
       {
         TReal = new TDataStd_Real;
@@ -150,9 +180,13 @@ bool BinMDataXtd_PatternStdDriver::Paste(const BinObjMgt_Persistent&       theSo
 
       // number of instances 2
       if (!(theSource >> aNb))
+      {
         return false;
+      }
       if (theRelocTable.IsBound(aNb))
+      {
         TInt = occ::down_cast<TDataStd_Integer>(theRelocTable.Find(aNb));
+      }
       else
       {
         TInt = new TDataStd_Integer;
@@ -177,17 +211,25 @@ void BinMDataXtd_PatternStdDriver::Paste(
   // signature
   int signature = aP->Signature();
   if (signature < 1 || signature > 5)
+  {
     signature = 0;
+  }
   theTarget << signature;
   if (signature == 0)
+  {
     return;
+  }
 
   // reversed flags
   int revFlags = 0;
   if (aP->Axis1Reversed())
+  {
     revFlags |= 1;
+  }
   if (aP->Axis2Reversed())
+  {
     revFlags |= 2;
+  }
   theTarget << revFlags;
 
   int aNb;

src/ApplicationFramework/TKBin/BinMDataXtd/BinMDataXtd_PositionDriver.cxx

@@ -49,18 +49,24 @@ bool BinMDataXtd_PositionDriver::Paste(const BinObjMgt_Persistent&       theSour
   double                         aValue;
   bool                           ok = theSource >> aValue;
   if (!ok)
+  {
     return ok;
+  }
   gp_Pnt aPosition(0., 0., 0.);
   aPosition.SetX(aValue);
 
   ok = theSource >> aValue;
   if (!ok)
+  {
     return ok;
+  }
   aPosition.SetY(aValue);
 
   ok = theSource >> aValue;
   if (!ok)
+  {
     return ok;
+  }
   aPosition.SetZ(aValue);
 
   anAtt->SetPosition(aPosition);