[clang-tools-extra] r205407 - Disable a few clang-tidy checks by default.

Daniel Jasper djasper at google.com
Wed Apr 2 01:27:14 PDT 2014


Author: djasper
Date: Wed Apr  2 03:27:12 2014
New Revision: 205407

URL: http://llvm.org/viewvc/llvm-project?rev=205407&view=rev
Log:
Disable a few clang-tidy checks by default.

The goal is to be able to run clang-tidy on LLVM files without further
configuration for now. Once llvm.org/PR19306 is addressed, we can add a
configuration file instead and choose other defaults.

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=205407&r1=205406&r2=205407&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp (original)
+++ clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp Wed Apr  2 03:27:12 2014
@@ -34,7 +34,11 @@ static cl::opt<std::string> Checks(
 static cl::opt<std::string> DisableChecks(
     "disable-checks",
     cl::desc("Regular expression matching the names of the checks to disable."),
-    cl::init("clang-analyzer-alpha.*"), cl::cat(ClangTidyCategory));
+    cl::init("(clang-analyzer-alpha.*" // To many false positives.
+             "|llvm-include-order"     // Not implemented yet.
+             "|llvm-namespace-comment" // Not complete.
+             "|google-.*"),            // Doesn't apply to LLVM.
+    cl::cat(ClangTidyCategory));
 static cl::opt<bool> Fix("fix", cl::desc("Fix detected errors if possible."),
                          cl::init(false), cl::cat(ClangTidyCategory));
 





More information about the cfe-commits mailing list