Merge pull request #278 from Exawind/ignore_macros_cognitive_complexity

The clang-tidy congnitive complexity check no longer expands macros.
Exawind · Oct 1, 2024 · 9b59dcf · 9b59dcf
2 parents 1f52662 + 141e68f
commit 9b59dcf
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.clang-tidy b/.clang-tidy
@@ -21,6 +21,8 @@ Checks: "bugprone*,
          -bugprone-easily-swappable-parameters,
          -readability-named-parameter,
 "
+CheckOptions:
+  readability-function-cognitive-complexity.IgnoreMacros: 'true'
 WarningsAsErrors: ''
 HeaderFilterRegex: '.*'
 FormatStyle:     none
diff --git a/tests/unit_tests/system/test_calculate.hpp b/tests/unit_tests/system/test_calculate.hpp
@@ -44,7 +44,6 @@ inline void CompareWithExpected(
     }
 }
 
-// NOLINTBEGIN(readability-function-cognitive-complexity)
 inline void CompareWithExpected(
     const Kokkos::View<const double*****>::host_mirror_type& result,
     const Kokkos::View<const double*****, Kokkos::HostSpace>& expected
@@ -61,5 +60,4 @@ inline void CompareWithExpected(
         }
     }
 }
-// NOLINTEND(readability-function-cognitive-complexity)
 }  // namespace openturbine::tests