GH Actions: Fix code formatter action (#51)

.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
-

src/System.cxx

@@ -143,4 +143,3 @@ void assertFileSystemType(const std::string& path, const std::set<std::string>&
 } // namespace System
 } // namespace Common
 } // namespace AliceO2
-

src/Thread.cxx

@@ -104,4 +104,3 @@ std::string Thread::getName()
 {
   return name;
 }
-

test/TestBasicThread.cxx

@@ -32,4 +32,3 @@ BOOST_AUTO_TEST_CASE(TestBasicThread)
   thread.join();
   BOOST_CHECK(number > 0);
 }
-

test/TestIommu.cxx

@@ -19,4 +19,3 @@ BOOST_AUTO_TEST_CASE(TestIsEnabled)
 {
   BOOST_CHECK_NO_THROW(AliceO2::Common::Iommu::isEnabled());
 }
-

test/TestSuffixNumber.cxx

@@ -46,4 +46,3 @@ BOOST_AUTO_TEST_CASE(TestFloatingPoint)
   sn.setNumber("-123");
   BOOST_CHECK_EQUAL(sn.getNumber(), -123);
 }
-

test/TestSuffixOption.cxx

@@ -45,4 +45,3 @@ BOOST_AUTO_TEST_CASE(TestSuffixOption)
   BOOST_CHECK(numberB.getNumber() == 123);
   BOOST_CHECK(numberC == -1000.0);
 }
-

test/TestSystem.cxx

@@ -87,4 +87,3 @@ BOOST_AUTO_TEST_CASE(TestIsFileSystemTypeAnyOf)
   BOOST_CHECK(System::isFileSystemTypeAnyOf("/sys", { "blahfs", "ext42" }).first == false);
 #endif
 }
-

test/testDaemon.cxx

@@ -20,4 +20,3 @@ int main(int argc, char* argv[])
   Daemon d(argc, argv);
   return d.run();
 }
-

test/testFifo.cxx

@@ -57,4 +57,3 @@ BOOST_AUTO_TEST_CASE(fifo_test)
 
   printf("fifoSz=%d sum=%d\n", fifoSz, sum2);
 }
-

test/testSimpleLog.cxx

@@ -21,10 +21,9 @@ int main()
 
   SimpleLog theLog;
   theLog.setLogFile("/tmp/test.log", 100, 4, 0);
-  for (int i=0; i<10; i++) {
-    theLog.info("test message %d",i);
+  for (int i = 0; i < 10; i++) {
+    theLog.info("test message %d", i);
   }
-  //sleep(10);
+  // sleep(10);
   return 0;
 }
-

test/testTimer.cxx

@@ -60,4 +60,3 @@ BOOST_AUTO_TEST_CASE(timer_test)
 
   BOOST_CHECK_EQUAL(success, 1);
 }
-