From 7d6bab2256f65c74f0328811c2b60be514763665 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Sat, 18 Mar 2023 14:23:36 +0100 Subject: [PATCH] Meta: Disable the misc-no-recursion check in clang-tidy We use recursive algorithms all over the place, and this check makes it hard to read those algorithms in CLion, since it draws squiggles under everything. --- .clang-tidy | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.clang-tidy b/.clang-tidy index 55e1f15028..ad2430864f 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -7,6 +7,7 @@ # cert-dcl37-c: Alias for bugprone-reserved-identifier # cert-dcl51-cpp: Alias for bugprone-reserved-identifier # cert-dcl21-cpp: No reference to this rule exists on Carnegie Mellon's SEI CERT C++ Confluence. And the suggestion is unusual +# misc-no-recursion: The project uses recursive algorithms in several places. # FIXME: misc-non-private-member-variables-in-classes: Audit uses of protected member variables to see if they really need to be protected # performance-noexcept-move-constructor: The project does not use exceptions, so there are no such optimizations available # performance-no-int-to-ptr: This rule flags every pointer to integer cast, which gets quite noisy. Should only be enabled on a case-by-case basis @@ -29,6 +30,7 @@ Checks: > -bugprone-macro-parentheses, -bugprone-reserved-identifier,-cert-dcl37-c,-cert-dcl51-cpp, -cert-dcl21-cpp, + -misc-no-recursion, -misc-non-private-member-variables-in-classes, -performance-noexcept-move-constructor, -performance-no-int-to-ptr,