Fixed compiler warnings

.gitea/workflows/action.yaml

@@ -0,0 +1,30 @@
+name: Test And Build
+on: [push]
+
+jobs:
+  Lint:
+    runs-on: linepics
+    steps:
+      - name: checkout repo
+        uses: actions/checkout@v4
+      - name: cppcheck
+        run: cppcheck --std=c++17 --addon=cert --addon=misc --error-exitcode=1 src/*.cpp
+      - name: formatting
+        run: clang-format --style=file --Werror --dry-run src/*.cpp src/*.h
+  Build:
+    runs-on: linepics
+    steps:
+      - name: checkout repo
+        uses: actions/checkout@v4
+        with:
+          submodules: 'true'
+      - run: |
+          pushd turboPmac
+          sed -i 's/ARCH_FILTER=.*/ARCH_FILTER=linux%/' Makefile
+          echo -e "\nIGNORE_SUBMODULES += sinqmotor" >> Makefile
+          make install
+          popd
+      - run: |
+          sed -i 's/ARCH_FILTER=.*/ARCH_FILTER=linux%/' Makefile
+          echo -e "\nIGNORE_SUBMODULES += turboPmac" >> Makefile
+          make install

Makefile

@@ -35,4 +35,5 @@ DBDS += turboPmac/sinqMotor/src/sinqMotor.dbd
 DBDS += turboPmac/src/turboPmac.dbd
 DBDS += src/beamShift.dbd
 
-USR_CFLAGS += -Wall -Wextra -Weffc++ -Wunused-result -Werror # -Wpedantic // Does not work because EPICS macros trigger warnings
+USR_CFLAGS += -Wall -Wextra -Wunused-result -Werror # -Wpedantic // Does not work because EPICS macros trigger warnings
+USR_CXXFLAGS += -Wall -Wextra -Wunused-result -Werror

src/beamShiftAxis.cpp

@@ -403,6 +403,12 @@ asynStatus beamShiftAxis::doMove(double position, int relative,
 
     // =========================================================================
 
+    // Suppress unused variables warnings
+    (void)relative;
+    (void)minVelocity;
+    (void)maxVelocity;
+    (void)acceleration;
+
     getAxisParamChecked(this, motorEnableRBV, &enabled);
     getAxisParamChecked(this, motorRecResolution, &motorRecResolution);
 
@@ -447,6 +453,9 @@ asynStatus beamShiftAxis::stop(double acceleration) {
 
     // =========================================================================
 
+    // Suppress unused variables warnings
+    (void)acceleration;
+
     return pC_->writeRead(axisNo(), "P150=8", response, 0);
 }
 
@@ -463,8 +472,15 @@ asynStatus beamShiftAxis::doReset() {
 /*
 This is a no-op.
 */
-asynStatus beamShiftAxis::doHome(double min_velocity, double max_velocity,
+asynStatus beamShiftAxis::doHome(double minVelocity, double maxVelocity,
                                  double acceleration, int forwards) {
+
+    // Suppress unused variables warnings
+    (void)minVelocity;
+    (void)maxVelocity;
+    (void)acceleration;
+    (void)forwards;
+
     return asynSuccess;
 }
 

src/beamShiftAxis.h

@@ -3,7 +3,7 @@
 #include "turboPmacAxis.h"
 #include "turboPmacController.h"
 
-class beamShiftAxis : public turboPmacAxis {
+class HIDDEN beamShiftAxis : public turboPmacAxis {
   public:
     /**
      * @brief Construct a new beamShiftAxis