Coding - Update Clang-format with more macroses (#677)

- Updates `.clang-format` to include `OCC_CATCH_SIGNALS`, `DEFINE_STANDARD_RTTIEXT`, and `Draw_Drawable3D_FACTORY` as statement macros
- Adds a GitHub action step to automatically remove empty lines after `Standard_DEPRECATED` macro declarations
- Reformats numerous files by removing blank lines after deprecated method declarations and improving macro formatting

Author: dpasukhi

.clang-format

@@ -36,6 +36,9 @@ UseTab: Never
 # OCCT specific settings
 StatementMacros:
   - Standard_FALLTHROUGH
+  - OCC_CATCH_SIGNALS
+  - DEFINE_STANDARD_RTTIEXT
+  - Draw_Drawable3D_FACTORY
   - Standard_DEPRECATED
 TypenameMacros:
   - Handle

.github/actions/clang-format-check/action.yml

@@ -56,6 +56,20 @@ runs:
           clang-format -i -style=file $_
         }
 
+    - name: Remove empty lines after Standard_DEPRECATED
+      if: steps.changed-files.outputs.has_files == 'true'
+      shell: pwsh
+      run: |
+        $files = Get-Content "changed_files.txt" | Where-Object { Test-Path $_ }
+        $files | ForEach-Object {
+          $content = Get-Content $_ -Raw
+          $pattern = '(Standard_DEPRECATED\("[a-z0-9\.,;\s\(\)_-]+"(?:[\s\n]+"[a-z0-9\.,;\s\(\)_-]+")?\)\n)\n+'
+          $newContent = $content -replace $pattern, '$1'
+          if ($content -ne $newContent) {
+            Set-Content -Path $_ -Value $newContent -NoNewline
+          }
+        }
+
     - name: Check git status
       id: git-check
       if: steps.changed-files.outputs.has_files == 'true'

src/ApplicationFramework/TKCAF/TNaming/TNaming_Identifier.cxx

@@ -264,7 +264,7 @@ Standard_Boolean IsImported(const Handle(TNaming_NamedShape)& NS)
   it.Next();
   if (!it.More())
     return 0;
-    // plus d un shape.
+  // plus d un shape.
 #ifdef OCCT_DEBUG
   std::cout << "WARNING IMPORTED" << std::endl;
 #endif

src/ApplicationFramework/TKCAF/TNaming/TNaming_Name.cxx

@@ -1498,7 +1498,7 @@ static Standard_Boolean FilterByNeighbourgs(const TDF_Label&                  L,
     TC = TopAbs_VERTEX;
   // clang-format off
   if (ShapeType == TopAbs_VERTEX) TC = TopAbs_VERTEX; // szy 31.03.10 - to process case when Candidate is of type Vertex
-    // clang-format on
+  // clang-format on
 
 #ifdef OCCT_DEBUG_FNB
   i    = 1;

src/ApplicationFramework/TKCDF/PCDM/PCDM_StorageDriver.cxx

@@ -49,7 +49,8 @@ void PCDM_StorageDriver::Write(const Handle(CDM_Document)&       aDocument,
   {
     try
     {
-      OCC_CATCH_SIGNALS Make(aDocument, thePersistentDocuments);
+      OCC_CATCH_SIGNALS
+      Make(aDocument, thePersistentDocuments);
     }
     catch (Standard_Failure const& anException)
     {

src/DataExchange/TKDESTEP/GeomToStep/GeomToStep_MakeBSplineCurveWithKnots.cxx

@@ -37,8 +37,7 @@
 //=============================================================================
 GeomToStep_MakeBSplineCurveWithKnots::GeomToStep_MakeBSplineCurveWithKnots(
   const Handle(Geom_BSplineCurve)& BS,
-  const StepData_Factors&          theLocalFactors)
-{
+  const StepData_Factors&          theLocalFactors){
 #define Array1OfPnt_gen TColgp_Array1OfPnt
 #include "GeomToStep_MakeBSplineCurveWithKnots_gen.pxx"
 #undef Array1OfPnt_gen

src/DataExchange/TKDESTEP/GeomToStep/GeomToStep_MakeBSplineCurveWithKnotsAndRationalBSplineCurve.cxx

@@ -38,12 +38,11 @@
 GeomToStep_MakeBSplineCurveWithKnotsAndRationalBSplineCurve::
   GeomToStep_MakeBSplineCurveWithKnotsAndRationalBSplineCurve(
     const Handle(Geom_BSplineCurve)& BS,
-    const StepData_Factors&          theLocalFactors)
-{
+    const StepData_Factors&          theLocalFactors){
 #define Array1OfPnt_gen TColgp_Array1OfPnt
 #include "GeomToStep_MakeBSplineCurveWithKnotsAndRationalBSplineCurve_gen.pxx"
 #undef Array1OfPnt_gen
-}
+  }
 
 //=============================================================================
 // Creation d' une bspline_curve_with_knots_and_rational_bspline_curve de

src/DataExchange/TKDESTEP/GeomToStep/GeomToStep_MakeLine.cxx

@@ -33,8 +33,7 @@
 //=============================================================================
 // Creation d' une line de prostep a partir d' une Lin de gp
 //=============================================================================
-GeomToStep_MakeLine::GeomToStep_MakeLine(const gp_Lin& L, const StepData_Factors& theLocalFactors)
-{
+GeomToStep_MakeLine::GeomToStep_MakeLine(const gp_Lin& L, const StepData_Factors& theLocalFactors){
 #define Vec_gen gp_Vec
 #include "GeomToStep_MakeLine_gen.pxx"
 #undef Vec_gen
@@ -44,8 +43,8 @@ GeomToStep_MakeLine::GeomToStep_MakeLine(const gp_Lin& L, const StepData_Factors
 // Creation d' une line de prostep a partir d' une Lin2d de gp
 //=============================================================================
 
-GeomToStep_MakeLine::GeomToStep_MakeLine(const gp_Lin2d& L, const StepData_Factors& theLocalFactors)
-{
+GeomToStep_MakeLine::GeomToStep_MakeLine(const gp_Lin2d&         L,
+                                         const StepData_Factors& theLocalFactors){
 #define Vec_gen gp_Vec2d
 #include "GeomToStep_MakeLine_gen.pxx"
 #undef Vec_gen

src/DataExchange/TKDESTEP/RWStepVisual/RWStepVisual_RWPresentationLayerUsage.cxx

@@ -34,7 +34,7 @@ void RWStepVisual_RWPresentationLayerUsage::ReadStep(
   if (!data->CheckNbParams(num, 2, ach, "presentation_layer_usage"))
     return;
 
-    // --- own fields
+  // --- own fields
 #include <StepVisual_PresentationLayerAssignment.hxx>
 #include <StepVisual_PresentationRepresentation.hxx>
   Handle(StepVisual_PresentationLayerAssignment) pla;

src/DataExchange/TKXCAF/XCAFDoc/XCAFDoc_VisMaterial.hxx

@@ -132,15 +132,13 @@ public:
   Standard_EXPORT void SetFaceCulling(Graphic3d_TypeOfBackfacingModel theFaceCulling);
 
   Standard_DEPRECATED("Deprecated method, FaceCulling() should be used instead")
-
   Standard_Boolean IsDoubleSided() const
   {
     return myFaceCulling == Graphic3d_TypeOfBackfacingModel_DoubleSided;
   }
 
   Standard_DEPRECATED("Deprecated method, SetFaceCulling() should be used "
                       "instead")
-
   void SetDoubleSided(Standard_Boolean theIsDoubleSided)
   {
     SetFaceCulling(theIsDoubleSided ? Graphic3d_TypeOfBackfacingModel_DoubleSided