[clang-tools-extra] r220865 - Shrink the set of checks enabled by default.
Alexander Kornienko
alexfh at google.com
Wed Oct 29 11:25:09 PDT 2014
Author: alexfh
Date: Wed Oct 29 13:25:09 2014
New Revision: 220865
URL: http://llvm.org/viewvc/llvm-project?rev=220865&view=rev
Log:
Shrink the set of checks enabled by default.
Summary:
Enable only compiler diagnostics and safe static analyzer checks by
default. Let the defaults be conservative and safe for an average project.
Reviewers: djasper
Reviewed By: djasper
Subscribers: cfe-commits
Differential Revision: http://reviews.llvm.org/D6027
Modified:
clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp
Modified: clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp?rev=220865&r1=220864&r2=220865&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp (original)
+++ clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp Wed Oct 29 13:25:09 2014
@@ -35,11 +35,10 @@ static cl::extrahelp ClangTidyHelp(
" option, command-line option takes precedence. The effective\n"
" configuration can be inspected using -dump-config.\n\n");
-const char DefaultChecks[] =
- "*," // Enable all checks, except these:
- "-clang-analyzer-alpha*," // Too many false positives.
- "-llvm-include-order," // Not implemented yet.
- "-google-*,"; // Doesn't apply to LLVM.
+const char DefaultChecks[] = // Enable these checks:
+ "clang-diagnostic-*," // * compiler diagnostics
+ "clang-analyzer-*," // * Static Analyzer checks
+ "-clang-analyzer-alpha*"; // * but not alpha checks: many false positives
static cl::opt<std::string>
Checks("checks", cl::desc("Comma-separated list of globs with optional '-'\n"
More information about the cfe-commits
mailing list