Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(774)

Unified Diff: tools/clang/plugins/FindBadConstructs.cpp

Issue 10853012: Add cc/ to banned_directories in chromium clang style checker (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: with command line flag guard Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/clang/plugins/ChromeClassTester.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/clang/plugins/FindBadConstructs.cpp
diff --git a/tools/clang/plugins/FindBadConstructs.cpp b/tools/clang/plugins/FindBadConstructs.cpp
index b9e1f5b1755c8649c607d9604b91a45de414e43e..0c41d7c54af2f955017075be4fde97251ae5cdc1 100644
--- a/tools/clang/plugins/FindBadConstructs.cpp
+++ b/tools/clang/plugins/FindBadConstructs.cpp
@@ -51,8 +51,9 @@ class FindBadConstructsConsumer : public ChromeClassTester {
FindBadConstructsConsumer(CompilerInstance& instance,
bool check_refcounted_dtors,
bool check_virtuals_in_implementations,
- bool check_inner_classes)
- : ChromeClassTester(instance, check_inner_classes),
+ bool check_inner_classes,
+ bool check_cc_directory)
+ : ChromeClassTester(instance, check_inner_classes, check_cc_directory),
check_refcounted_dtors_(check_refcounted_dtors),
check_virtuals_in_implementations_(check_virtuals_in_implementations) {
}
@@ -397,7 +398,8 @@ class FindBadConstructsAction : public PluginASTAction {
FindBadConstructsAction()
: check_refcounted_dtors_(true),
check_virtuals_in_implementations_(true),
- check_inner_classes_(false) {
+ check_inner_classes_(false),
+ check_cc_directory_(false) {
}
protected:
@@ -406,7 +408,7 @@ class FindBadConstructsAction : public PluginASTAction {
llvm::StringRef ref) {
return new FindBadConstructsConsumer(
instance, check_refcounted_dtors_, check_virtuals_in_implementations_,
- check_inner_classes_);
+ check_inner_classes_, check_cc_directory_);
}
virtual bool ParseArgs(const CompilerInstance& instance,
@@ -420,6 +422,8 @@ class FindBadConstructsAction : public PluginASTAction {
check_virtuals_in_implementations_ = false;
} else if (args[i] == "check-inner-classes") {
check_inner_classes_ = true;
+ } else if (args[i] == "check-cc-directory") {
+ check_cc_directory_ = true;
} else {
parsed = false;
llvm::errs() << "Unknown argument: " << args[i] << "\n";
@@ -433,6 +437,7 @@ class FindBadConstructsAction : public PluginASTAction {
bool check_refcounted_dtors_;
bool check_virtuals_in_implementations_;
bool check_inner_classes_;
+ bool check_cc_directory_;
};
} // namespace
« no previous file with comments | « tools/clang/plugins/ChromeClassTester.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698