From 325866c7df1822306f230650190f80a9052c5934 Mon Sep 17 00:00:00 2001 From: Alexander Zhang Date: Tue, 22 Nov 2022 23:52:10 -0800 Subject: [PATCH] Disable irrelevant clang-tidy checks Previously, the clang-tidy configuration enables all checks and then disables a few undesired ones. This enables a lot of checks that are specific to certain kinds of code and aren't relevant for this project. For example, I noticed several errors from checks specific to LLVM libc. It makes more sense to enable only the checks that we want instead of enabling everything and then disabling the checks that we don't want. --- .clang-tidy | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.clang-tidy b/.clang-tidy index 6f50d123a..c52d418ce 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -1,5 +1,5 @@ --- -Checks: '*,readability-*,-google-runtime-references,-modernize-use-trailing-return-type,-hicpp-signed-bitwise,-fuchsia*,-modernize-use-nodiscard' +Checks: 'bugprone-*,cert-*,concurrency-*,cppcoreguidelines-*,misc-*,modernize-*,-modernize-use-trailing-return-type,-modernize-use-nodiscard,performance-*,portability-*,readability-*' WarningsAsErrors: '' HeaderFilterRegex: '' AnalyzeTemporaryDtors: false