Add CPPCHECK and Clang Tidy in CMake

.clang-tidy

@@ -0,0 +1,34 @@
+---
+Checks: '*,
+-clang-diagnostic-error,
+-clang-diagnostic-unused-variable,
+-cppcoreguidelines-avoid-magic-numbers,
+-cppcoreguidelines-non-private-member-variables-in-classes,
+-cppcoreguidelines-pro-bounds-array-to-pointer-decay,
+-cppcoreguidelines-pro-bounds-pointer-arithmetic,
+-cppcoreguidelines-pro-type-cstyle-cast,
+-cppcoreguidelines-pro-type-vararg,
+-cppcoreguidelines-special-member-functions,
+-fuchsia-default-arguments-calls,
+-fuchsia-default-arguments-declarations,
+-google-readability-avoid-underscore-in-googletest-name,
+-google-readability-namespace-comments,
+-google-readability-todo,
+-google-runtime-references,
+-hicpp-avoid-c-arrays,
+-hicpp-no-array-decay,
+-hicpp-no-malloc,
+-hicpp-special-member-functions,
+-hicpp-vararg,
+-llvm-header-guard,
+-llvm-namespace-comment,
+-llvmlibc-*,
+-misc-non-private-member-variables-in-classes,
+-modernize-avoid-c-arrays,
+-modernize-use-trailing-return-type,
+-readability-else-after-return,
+-readability-implicit-bool-conversion,
+-readability-convert-member-functions-to-static,
+'
+HeaderFilterRegex: '.*(include/gaussianblur).*'
+ExcludeHeaderFilterRegex: '.*(external/googletest/googletest/include|include/pffft_pommier).*'

.cppcheck-suppressions

@@ -0,0 +1,3 @@
+*:*/stb_image*
+*:*/pffft*
+*:*/googletest*

CMakeLists.txt

@@ -28,9 +28,17 @@ endif()
 # If compiling to WebAssembly
 if(WASM)
     message(STATUS "Compiling to WebAssembly")
-    if(NOT DEFINED CMAKE_TOOLCHAIN_FILE)
-        set(CMAKE_TOOLCHAIN_FILE "$ENV{HOME}/emsdk/upstream/emscripten/cmake/Modules/Platform/Emscripten.cmake")
+    if(NOT CMAKE_TOOLCHAIN_FILE OR NOT CMAKE_TOOLCHAIN_FILE MATCHES "Emscripten.cmake")
+        find_program(EMCC_EXE emcc)
+        if (EMCC_EXE)
+            message ("-- Found Emscripten")
+            get_filename_component(EMSDK_DIR ${EMCC_EXE} DIRECTORY)
+            set(CMAKE_TOOLCHAIN_FILE "${EMSDK_DIR}/cmake/Modules/Platform/Emscripten.cmake")
+        else()
+            message(FATAL_ERROR "-- Emscripten not found! Aborting...")
+        endif()
     endif()
+    message(STATUS "Using Emscripten toolchain file: ${CMAKE_TOOLCHAIN_FILE}")
     set(CMAKE_EXECUTABLE_SUFFIX ".wasm")
 endif()
 
@@ -60,15 +68,39 @@ if(APPLE AND NOT WASM)
     set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -framework Accelerate")
 endif()
 
+# Cpp check for static analysis
+find_program(CPPCHECK_EXE cppcheck)
+if (CPPCHECK_EXE)
+    message("-- Program cppcheck found: ${CPPCHECK_EXE}")
+    set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+    add_custom_target(cppcheck
+            COMMAND "${CPPCHECK_EXE}" --std=c++${CMAKE_CXX_STANDARD} "--suppressions-list=${CMAKE_SOURCE_DIR}/.cppcheck-suppressions" --enable=all "--project=${CMAKE_BINARY_DIR}/compile_commands.json")
+else ()
+    message("-- Program cppcheck NOT found!")
+endif ()
+
+# Clang tidy
+if (CLANG_TIDY_ENABLED)
+    find_program(CLANG_TIDY_EXE clang-tidy)
+    if (CLANG_TIDY_EXE)
+        message("-- Program clang-tidy found: ${CLANG_TIDY_EXE}")
+        set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+        set(CMAKE_C_CLANG_TIDY "${CLANG_TIDY_EXE}" --quiet "--config-file=${CMAKE_SOURCE_DIR}/.clang-tidy" -p "${CMAKE_BINARY_DIR}/compile_commands.json")
+        set(CMAKE_CXX_CLANG_TIDY "${CMAKE_C_CLANG_TIDY}")
+    else ()
+        message("-- Program clang-tidy NOT found!")
+    endif ()
+endif ()
+
 # Project name
 project(GaussianBlur)
 include_directories(include)
 
 # Add library
 if(WASM)
-    add_library(GaussianBlurLib STATIC "include/pffft_pommier/pffft.c" "src/gaussianblur.cpp")
+    add_library(GaussianBlurLib STATIC "${CMAKE_SOURCE_DIR}/include/pffft_pommier/pffft.c" "${CMAKE_SOURCE_DIR}/src/gaussianblur.cpp")
 else()
-    add_library(GaussianBlurLib "include/pffft_pommier/pffft.c" "src/gaussianblur.cpp")
+    add_library(GaussianBlurLib "${CMAKE_SOURCE_DIR}/include/pffft_pommier/pffft.c" "${CMAKE_SOURCE_DIR}/src/gaussianblur.cpp")
 endif()
 
 # Set properties for the library
@@ -77,7 +109,7 @@ set_target_properties(GaussianBlurLib PROPERTIES OUTPUT_NAME "Gaussianblur")
 if(WITH_EXAMPLES)
     message(STATUS "Building examples")
     if(WASM)
-        add_executable(GaussianBlur "examples/wasm/main.cpp")
+        add_executable(GaussianBlur "${CMAKE_SOURCE_DIR}/examples/wasm/main.cpp")
         set(WASM_COMMON_LINK_OPTIONS 
             "SHELL: --closure 1"
             "-sALLOW_MEMORY_GROWTH=1"
@@ -104,7 +136,7 @@ if(WITH_EXAMPLES)
         # Install the WASM file
         install(FILES "${CMAKE_BINARY_DIR}/GaussianBlur.wasm" DESTINATION bin)
     else()
-        add_executable(GaussianBlur "examples/desktop/main.cpp")
+        add_executable(GaussianBlur "${CMAKE_SOURCE_DIR}/examples/desktop/main.cpp")
     endif()
 
     target_link_libraries(GaussianBlur GaussianBlurLib)
@@ -127,12 +159,17 @@ if(WITH_TESTS)
     set(BUILD_SHARED_LIBS_SAVE ${BUILD_SHARED_LIBS})
     set(BUILD_SHARED_LIBS OFF)
     add_subdirectory(external/googletest)
+
+    # Skip clang-tidy for Google Test
+    set_target_properties(gtest PROPERTIES CXX_CLANG_TIDY "" C_CLANG_TIDY "")
+    set_target_properties(gtest_main PROPERTIES CXX_CLANG_TIDY "" C_CLANG_TIDY "")
+
     set(BUILD_SHARED_LIBS ${BUILD_SHARED_LIBS_SAVE})
 
     include_directories(${gtest_SOURCE_DIR}/include ${gtest_SOURCE_DIR})
 
     # Add test executable
-    add_executable(GaussianBlurTests tests/test_gaussianblur.cpp)
+    add_executable(GaussianBlurTests ${CMAKE_SOURCE_DIR}/tests/test_gaussianblur.cpp)
     target_link_libraries(GaussianBlurTests GaussianBlurLib gtest_main)
     add_test(NAME GaussianBlurTests COMMAND GaussianBlurTests)
     install(TARGETS GaussianBlurTests DESTINATION bin)

Dockerfile

@@ -2,8 +2,8 @@ FROM debian:bookworm-slim AS base
 
 # Install dependencies
 RUN apt-get -qq update; \
-    apt-get install -qqy --no-install-recommends \
-        gnupg2 wget ca-certificates apt-transport-https curl unzip make cmake xz-utils
+    apt-get install -qqy --no-install-recommends gnupg2 wget ca-certificates \
+    apt-transport-https curl unzip make cmake xz-utils cppcheck
 
 # Install LLVM
 RUN echo "deb https://apt.llvm.org/bookworm llvm-toolchain-bookworm-16 main" \

src/gaussianblur.cpp

@@ -15,7 +15,7 @@ int gaussian_window(const float sigma, const int max_width = 0) {
   // bigger than it
 
   const float radius = sigma * sqrt(2 * log(255)) - 1;
-  int width = radius * 2 + 0.5f;
+  int width = radius * 2 + 0.5F;
   if (max_width) width = std::min(width, max_width);
 
   if (width % 2 == 0) ++width;
@@ -34,19 +34,19 @@ void get_gaussian(T &kernel, const float sigma, int width = 0,
 
   kernel.resize(FFT_length ? FFT_length : width);
 
-  const float mid_w = (width - 1) / 2.0f;
-  const float s = 2.0f * sigma * sigma;
+  const float mid_w = (width - 1) / 2.0F;
+  const float s = 2.0F * sigma * sigma;
 
   int i = 0;
 
   for (float y = -mid_w; y <= mid_w; ++y, ++i)
     kernel.at(i) = (exp(-(y * y) / s)) / (std::numbers::pi_v<float> * s);
 
   const float sum =
-      1.0f / std::accumulate(kernel.begin(), kernel.begin() + width, 0.0f);
+      1.0F / std::accumulate(kernel.begin(), kernel.begin() + width, 0.0F);
 
   std::transform(kernel.begin(), kernel.begin() + width, kernel.begin(),
-                 [&sum](auto &i) { return i * sum; });
+                 [&sum](const auto &i) { return i * sum; });
 
   // fit the kernel in the FFT_length and shift the center to avoid circular
   // convolution
@@ -248,8 +248,8 @@ void pffft(const ImgGeom image_geometry, const KernelDFT kernelDFT,
       std::chrono::steady_clock::now();
   const int maxsize =
       std::max(kernelDFT.kerf_1D_row.size(), kernelDFT.kerf_1D_col.size());
-  const float divisor_col = 1.0f / kernelDFT.kerf_1D_col.size();
-  const float divisor_row = 1.0f / kernelDFT.kerf_1D_row.size();
+  const float divisor_col = 1.0F / kernelDFT.kerf_1D_col.size();
+  const float divisor_row = 1.0F / kernelDFT.kerf_1D_row.size();
 
   AlignedVector<float> tmp;
   tmp.reserve(maxsize);

tests/test_gaussianblur.cpp

@@ -11,7 +11,7 @@
 // Test case for prepare_kernel_DFT
 TEST(GaussianBlurTest, PrepareKernelDFT) {
   ImgGeom image_geom = {4, 4, 3};  // 4x4 image with 3 channels (RGB)
-  float sigma = 2.0f;
+  float sigma = 2.0F;
 
   // Prepare the kernel DFT
   KernelDFT kernel_dft = gaussianblur::prepare_kernel_DFT(image_geom, sigma);
@@ -40,7 +40,7 @@ TEST(GaussianBlurTest, BasicTestRGB) {
   ImgGeom image_geom = {3, 3, 3};  // 3x3 image with 3 channels (RGB)
   Image image = {image_data, image_geom};
 
-  float sigma = 3.0f;
+  float sigma = 3.0F;
   bool apply_to_alpha = false;
 
   // Calculate the average color and variance of the original image
@@ -72,7 +72,7 @@ TEST(GaussianBlurTest, InvalidChannelCount) {
   ImgGeom image_geom = {3, 3, 2};  // 3x3 image with 2 channels
   Image image = {image_data, image_geom};
 
-  float sigma = 3.0f;
+  float sigma = 3.0F;
   bool apply_to_alpha = false;
 
   gaussianblur::gaussianblur(image, sigma, apply_to_alpha);
@@ -95,7 +95,7 @@ TEST(GaussianBlurTest, BasicTestRGBAWithoutAlpha) {
   ImgGeom image_geom = {3, 3, 4};  // 3x3 image with 4 channels (RGBA)
   Image image = {image_data, image_geom};
 
-  float sigma = 3.0f;
+  float sigma = 3.0F;
   bool apply_to_alpha = false;
 
   // Calculate the average color and variance of the original image
@@ -139,7 +139,7 @@ TEST(GaussianBlurTest, StressTest) {
   ImgGeom image_geom = {width, height, channels};
   Image image = {image_data, image_geom};
 
-  float sigma = 5.0f;
+  float sigma = 5.0F;
   bool apply_to_alpha = true;
 
   // Measure the time taken to apply Gaussian blur
@@ -177,7 +177,7 @@ TEST(GaussianBlurTest, BasicTestRGBAWithAlpha) {
   ImgGeom image_geom = {3, 3, 4};  // 3x3 image with 4 channels (RGBA)
   Image image = {image_data, image_geom};
 
-  float sigma = 3.0f;
+  float sigma = 3.0F;
   bool apply_to_alpha = true;
 
   // Calculate the average color and variance of the original image
@@ -207,14 +207,14 @@ TEST(GaussianBlurTest, BasicTestRGBAWithAlpha) {
 // Test case for flip_block
 TEST(HelpersTest, FlipBlock) {
   // Create a simple 2x2 block
-  std::vector<float> input = {1.0f, 2.0f, 3.0f, 4.0f};
+  std::vector<float> input = {1.0F, 2.0F, 3.0F, 4.0F};
   std::vector<float> output(4);
 
   // Flip the block
   flip_block<1>(input.data(), output.data(), 2, 2);
 
   // Check the results
-  std::vector<float> expected_output = {1.0f, 3.0f, 2.0f, 4.0f};
+  std::vector<float> expected_output = {1.0F, 3.0F, 2.0F, 4.0F};
   ASSERT_EQ(output, expected_output);
 }