Introduce clang-tidy and fix diagnostics

Signed-off-by: Philipp Jungkamp <[email protected]>

Author: pjungkamp

.clang-tidy

@@ -0,0 +1,10 @@
+# SPDX-FileCopyrightText: 2025 Institute for Automation of Complex Power Systems, RWTH Aachen University
+# SPDX-License-Identifier: Apache-2.0
+
+Checks: >
+  -*,
+  modernize-use-override,
+  misc-const-correctness,
+WarningsAsErrors: '*'
+HeaderFilterRegex: villas/.*
+FormatStyle: file

CMakeLists.txt

@@ -53,7 +53,13 @@ include(FindSymbol)
 include(CMakeDependentOption)
 
 add_definitions(-D_POSIX_C_SOURCE=200809L -D_GNU_SOURCE)
-add_compile_options(-Wall -Wno-unknown-pragmas -fdiagnostics-color=auto)
+add_compile_options(
+    -Wall
+    -Wno-unknown-pragmas
+    -Wno-vla-cxx-extension
+    -Wno-unused-command-line-argument
+    -fdiagnostics-color=auto
+)
 
 # Check OS
 check_include_file("sys/eventfd.h" HAS_EVENTFD)

clients/shmem/villas-shmem.cpp

@@ -34,7 +34,7 @@ class Shmem : public Tool {
 protected:
   std::atomic<bool> stop;
 
-  void usage() {
+  void usage() override {
     std::cout
         << "Usage: villas-test-shmem WNAME VECTORIZE" << std::endl
         << "  WNAME     name of the shared memory object for the output queue"
@@ -47,9 +47,9 @@ class Shmem : public Tool {
     printCopyright();
   }
 
-  void handler(int, siginfo_t *, void *) { stop = true; }
+  void handler(int, siginfo_t *, void *) override { stop = true; }
 
-  int main() {
+  int main() override {
     int ret, readcnt, writecnt, avail;
 
     struct ShmemInterface shm;
@@ -62,9 +62,9 @@ class Shmem : public Tool {
       return 1;
     }
 
-    std::string wname = argv[1];
-    std::string rname = argv[2];
-    int vectorize = atoi(argv[3]);
+    std::string const wname = argv[1];
+    std::string const rname = argv[2];
+    int const vectorize = atoi(argv[3]);
 
     ret = shmem_int_open(wname.c_str(), rname.c_str(), &shm, &conf);
     if (ret < 0)
@@ -87,7 +87,7 @@ class Shmem : public Tool {
         outsmps[i]->sequence = insmps[i]->sequence;
         outsmps[i]->ts = insmps[i]->ts;
 
-        int len = MIN(insmps[i]->length, outsmps[i]->capacity);
+        int const len = MIN(insmps[i]->length, outsmps[i]->capacity);
         memcpy(outsmps[i]->data, insmps[i]->data, SAMPLE_DATA_LENGTH(len));
 
         outsmps[i]->length = len;

common/include/villas/cpuset.hpp

@@ -75,7 +75,7 @@ class CpuSet {
   size_t size() const { return sz; }
 
   CpuSet operator~() {
-    CpuSet full = UINTMAX_MAX;
+    CpuSet const full = UINTMAX_MAX;
 
     return full ^ *this;
   }

common/include/villas/dsp/window_cosine.hpp

@@ -27,7 +27,7 @@ class CosineWindow : public Window<T> {
 
   T correctionFactor;
 
-  virtual T filter(T in, size_type i) const { return in * coefficients[i]; }
+  T filter(T in, size_type i) const override { return in * coefficients[i]; }
 
 public:
   CosineWindow(double a0, double a1, double a2, double a3, double a4,

common/include/villas/exceptions.hpp

@@ -92,7 +92,7 @@ class ConfigError : public std::runtime_error {
   }
 
 public:
-  ~ConfigError() {
+  ~ConfigError() override {
     if (msg)
       free(msg);
   }
@@ -132,12 +132,12 @@ class ConfigError : public std::runtime_error {
   }
 
   std::string docUri() const {
-    std::string baseUri = "https://villas.fein-aachen.org/doc/jump?";
+    std::string const baseUri = "https://villas.fein-aachen.org/doc/jump?";
 
     return baseUri + id;
   }
 
-  virtual const char *what() const noexcept { return msg; }
+  const char *what() const noexcept override { return msg; }
 };
 
 } // namespace villas

common/include/villas/graph/directed.hpp

@@ -100,7 +100,7 @@ class DirectedGraph {
   EdgeIdentifier addDefaultEdge(VertexIdentifier fromVertexId,
                                 VertexIdentifier toVertexId) {
     // Create a new edge
-    std::shared_ptr<EdgeType> edge(new EdgeType);
+    std::shared_ptr<EdgeType> const edge(new EdgeType);
 
     return addEdge(edge, fromVertexId, toVertexId);
   }

common/include/villas/kernel/devices/driver.hpp

@@ -24,6 +24,7 @@ class Driver {
   virtual std::string name() const = 0;
   virtual void override(const Device &device) const = 0;
   virtual void unbind(const Device &device) const = 0;
+  virtual ~Driver() {}
 };
 
 } // namespace devices

common/include/villas/kernel/vfio_device.hpp

@@ -11,8 +11,6 @@
 
 #pragma once
 
-#include <list>
-#include <memory>
 #include <string>
 #include <vector>
 
@@ -99,7 +97,7 @@ class Device {
   std::vector<void *> mappings;
 
   // libpci handle of the device
-  const kernel::devices::PciDevice *pci_device;
+  [[maybe_unused]] const kernel::devices::PciDevice *pci_device;
 
   Logger log;
 };

common/include/villas/memory.hpp

@@ -116,7 +116,8 @@ template <typename DerivedAllocator> class BaseAllocator {
       : memoryAddrSpaceId(memoryAddrSpaceId) {
     // CRTP
     derivedAlloc = static_cast<DerivedAllocator *>(this);
-    std::string loggerName = fmt::format("memory:", derivedAlloc->getName());
+    std::string const loggerName =
+        fmt::format("memory:", derivedAlloc->getName());
     logger = Log::get(loggerName);
 
     // Default deallocation callback
@@ -136,7 +137,6 @@ template <typename DerivedAllocator> class BaseAllocator {
   }
 
   virtual std::unique_ptr<MemoryBlock, MemoryBlock::deallocator_fn>
-
   allocateBlock(size_t size) = 0;
 
   template <typename T> MemoryAccessor<T> allocate(size_t num) {
@@ -152,7 +152,7 @@ template <typename DerivedAllocator> class BaseAllocator {
     // Check if the allocated memory is really accessible by writing to the
     // allocated memory and reading back. Exponentially increase offset to
     // speed up testing.
-    MemoryAccessor<volatile uint8_t> byteAccessor(*mem);
+    MemoryAccessor<volatile uint8_t> const byteAccessor(*mem);
     size_t idx = 0;
     for (int i = 0; idx < mem->getSize(); i++, idx = (1 << i)) {
       auto val = static_cast<uint8_t>(i);
@@ -166,6 +166,8 @@ template <typename DerivedAllocator> class BaseAllocator {
     return MemoryAccessor<T>(std::move(mem));
   }
 
+  virtual ~BaseAllocator() {}
+
 protected:
   void insertMemoryBlock(const MemoryBlock &mem) {
     auto &mm = MemoryManager::get();
@@ -217,8 +219,8 @@ class LinearAllocator : public BaseAllocator<LinearAllocator> {
 
   std::string getName() const;
 
-  virtual std::unique_ptr<MemoryBlock, MemoryBlock::deallocator_fn>
-  allocateBlock(size_t size);
+  std::unique_ptr<MemoryBlock, MemoryBlock::deallocator_fn>
+  allocateBlock(size_t size) override;
 
 private:
   static constexpr size_t alignBytes = sizeof(uintptr_t);
@@ -247,7 +249,7 @@ class HostRam {
     std::string getName() const { return "HostRamAlloc"; }
 
     std::unique_ptr<MemoryBlock, MemoryBlock::deallocator_fn>
-    allocateBlock(size_t size);
+    allocateBlock(size_t size) override;
   };
 
   static HostRamAllocator &getAllocator() {
@@ -264,7 +266,7 @@ class HostDmaRam {
   public:
     HostDmaRamAllocator(int num);
 
-    virtual ~HostDmaRamAllocator();
+    ~HostDmaRamAllocator() override;
 
     std::string getName() const { return getUdmaBufName(num); }