From b1ecf05b3303fda057dcd87e46e75cb779d97f7d Mon Sep 17 00:00:00 2001 From: Heejin Ahn Date: Thu, 23 May 2019 11:57:05 -0700 Subject: Disable clang static analyzer in clang-tidy (#2136) clang-tidy by default enables two groups of checks: clang-diagnostic-* and clang-analyzer-*. Between the two, clang-analyzer is a static analyzer, but it seems to often produces false warnings. For example, when you write `object->someFunction()`, if it is not sure if object is not a NULL, i.e., if there's no assert(object) before, it produces a warning. This is sometimes annoying and inserting `assert` everywhere does not seem to be very appealing. I also noticed LLVM and V8 also disabled this analyzer. `-*` disables all checks. Then I re-enabled clang-diagnostic and readability-braces-around-statements. --- .clang-tidy | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to '.clang-tidy') diff --git a/.clang-tidy b/.clang-tidy index dbc7730e6..c8e78f1c0 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -1,4 +1,4 @@ -Checks: 'readability-braces-around-statements' +Checks: '-*,clang-diagnostic-*,readability-braces-around-statements' CheckOptions: - key: readability-identifier-naming.ClassCase value: CamelCase -- cgit v1.2.3