GH Actions: Fix code formatter action (#51)

Author: singiamtel

.github/workflows/format.yml

@@ -3,24 +3,26 @@ name: Clang format
 on: [pull_request]
 
 jobs:
-  clang-format-8:
-    runs-on: ubuntu-latest
+  clang-format:
+    runs-on: ubuntu-24.04
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v4
       with:
         ref: ${{ github.event.pull_request.head.sha }}
-    - name: Run clang-foramt on changed files
+    - name: Run clang-format on changed files
       run: |
         set -x
+        sudo apt install clang-format
+        git config --global --add safe.directory '*'
         git fetch origin ${{ github.event.pull_request.base.ref }}
         git fetch origin pull/${{ github.event.pull_request.number }}/head:${{ github.event.pull_request.head.ref }}
         BASE_COMMIT=$(git rev-parse ${{ github.event.pull_request.base.sha }})
-        COMMIT_FILES=$(git diff --name-only ${BASE_COMMIT} | grep -i -v LinkDef)
-        RESULT_OUTPUT=$(git-clang-format-8 --commit ${BASE_COMMIT} --diff --binary $(which clang-format-8) ${COMMIT_FILES})
+        COMMIT_FILES=$(git diff --name-only "${BASE_COMMIT}" | grep -i -v LinkDef)
+        RESULT_OUTPUT=$(git clang-format --commit "${BASE_COMMIT}" --diff --binary "$(which clang-format)" -- "${COMMIT_FILES}")
         if [ "$RESULT_OUTPUT" == "no modified files to format" ] || [ "$RESULT_OUTPUT" == "clang-format did not modify any files" ]; then
           exit 0
         else
-          git-clang-format-8 --commit $BASE_COMMIT --diff --binary $(which clang-format-8)
+          git clang-format --commit "$BASE_COMMIT" --diff --binary "$(which clang-format)"
           echo "$RESULT_OUTPUT"
           exit 1
-        fi  
+        fi

src/Configuration.cxx

@@ -113,7 +113,7 @@ void getValueSpecialization(void)
   vString = f.getValue<std::string>(s);
 }
 
-//template void ConfigFile::getValue<float>(std::string, float&);
+// template void ConfigFile::getValue<float>(std::string, float&);
 
 // http://stackoverflow.com/questions/4586768/how-to-iterate-a-boost-property-tree
 string indent(int level)
@@ -223,4 +223,3 @@ ConfigFileBrowser::Iterator ConfigFileBrowser::end()
 {
   return Iterator(this, ptPtr->end(), ptPtr->end());
 }
-

src/Daemon.cxx

@@ -278,4 +278,3 @@ Daemon::LoopStatus Daemon::doLoop()
 
 // todo
 // boost exceptions?
-

src/Exception.cxx

@@ -38,4 +38,3 @@ const char* Exception::what() const noexcept
 
 } // namespace Common
 } // namespace AliceO2
-

src/Iommu.cxx

@@ -33,4 +33,3 @@ bool isEnabled()
 } // namespace Iommu
 } // namespace Common
 } // namespace AliceO2
-

src/LineBuffer.cxx

@@ -112,5 +112,4 @@ int LineBuffer::getNextLine(std::string& nextLine)
   return 0;
 }
 
-//void LineBuffer::appendString(const char *s) {}
-
+// void LineBuffer::appendString(const char *s) {}

src/MemPool.cxx

@@ -91,7 +91,7 @@ void* MemPool::getPage()
     int newIx = (lastPageIndexGet + 1 + i) % numberOfPages;
     if (!pageIsUsed[newIx].test_and_set()) {
       lastPageIndexGet = newIx;
-      //printf("getPage() scan => %d\n",j);
+      // printf("getPage() scan => %d\n",j);
       return pageTable[newIx];
     }
     j++;
@@ -109,7 +109,7 @@ void MemPool::releasePage(void* pagePtr)
     if (pagePtr == pageTable[pageIx]) {
       pageIsUsed[pageIx].clear();
       lastPageIndexRelease = pageIx;
-      //printf("realeasePage() scan => %d\n",j);
+      // printf("realeasePage() scan => %d\n",j);
       return;
     }
     j++;
@@ -120,4 +120,3 @@ int MemPool::getPageSize()
 {
   return pageSize;
 }
-

src/Program.cxx

@@ -141,4 +141,3 @@ int Program::execute(int argc, char** argv)
 
 } // namespace Common
 } // namespace AliceO2
-

src/SimpleLog.cxx

@@ -125,7 +125,7 @@ int SimpleLog::Impl::logV(SimpleLog::Impl::Severity s, const char* message, va_l
     } else if (s == Severity::Warning) {
       ix += snprintf(&buffer[ix], len - ix, "Warning - ");
     } else {
-      //ix+=snprintf(&buffer[ix], len-ix, "");
+      // ix+=snprintf(&buffer[ix], len-ix, "");
     }
   }
 
@@ -410,4 +410,3 @@ void SimpleLog::Impl::rotate()
 }
 
 /// \todo: thread to flush output every 1 second
-

src/SuffixNumber.cxx

@@ -50,4 +50,3 @@ const std::vector<std::pair<std::string, const size_t>>& get()
 } // namespace _SuffixNumberTable
 } // namespace Common
 } // namespace AliceO2
-