From 767d34e3bdddef6c1871006dd0a2d06a4e1bcd5d Mon Sep 17 00:00:00 2001 From: Aaron Ballman Date: Thu, 27 May 2021 10:37:33 -0400 Subject: [PATCH] Disable misc-no-recursion checking in Clang We currently enable misc-no-recursion, but Clang uses recursion intentionally in a fair number of places (like RecursiveASTVisitor). Disabling this check reduces a noise in reviews that add new AST nodes, like https://reviews.llvm.org/D103112#2780747 which has five CI warnings that the author can do nothing about. --- clang/.clang-tidy | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/clang/.clang-tidy b/clang/.clang-tidy index 5b425a7..f517e92 100644 --- a/clang/.clang-tidy +++ b/clang/.clang-tidy @@ -1,4 +1,4 @@ -Checks: '-*,clang-diagnostic-*,llvm-*,misc-*,-misc-unused-parameters,-misc-non-private-member-variables-in-classes,-readability-identifier-naming' +Checks: '-*,clang-diagnostic-*,llvm-*,misc-*,-misc-unused-parameters,-misc-non-private-member-variables-in-classes,-readability-identifier-naming,-misc-no-recursion' # Note that the readability-identifier-naming check is disabled, there are too # many violations in the codebase and they create too much noise in clang-tidy # results. -- 2.7.4