Merge pull request #11 from urboob21/dev_branch

update: Enable cppcheck inline suspressions

Author: urbytes21

.github/workflows/cpp-build-test-coverage.yml

@@ -46,13 +46,15 @@ jobs:
       # Step 2: Run Cppcheck (Static Analysis)
       # - Scans for common C++ issues (style, memory, logic)
       # - You can adjust `--enable=` options as needed
+      # - https://cppcheck.sourceforge.io/manual.pdf
       # -------------------------------------------------------
       - name: Run static analysis with Cppcheck
         run: |
           echo "Running Cppcheck..."
           cppcheck --enable=warning,style,performance,portability \
                    --inconclusive \
                    --quiet \
+                   --inline-suppr \
                    --error-exitcode=1 \
                    ./src ./include
 

src/core/datatypes/class/RoleOfThreeFiveZero.cpp

@@ -1,3 +1,5 @@
+// cppcheck-suppress-file [noCopyConstructor, noOperatorEq, accessMoved]
+
 #include <cstring>
 #include <iostream>
 #include <utility>

src/core/datatypes/class/SallowDeepCopying.cpp

@@ -1,3 +1,5 @@
+// cppcheck-suppress-file [noCopyConstructor, noOperatorEq, unsafeClassCanLeak]
+
 #include <iostream>
 
 namespace
@@ -35,12 +37,13 @@ namespace
                 }
             }
 
-            int getPtr()
+            int getPtr() const
             {
                 if (ptr != nullptr)
                 {
                     return *ptr;
                 }
+                return 0;
             }
 
             // Implicit copy constructor: Model a{b};
@@ -149,12 +152,13 @@ namespace
                 }
             }
 
-            int getPtr()
+            int getPtr() const
             {
                 if (ptr != nullptr)
                 {
                     return *ptr;
                 }
+                return 0;
             }
         };