OLD | NEW |
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "FindBadConstructsAction.h" | 5 #include "FindBadConstructsAction.h" |
6 | 6 |
7 #include "clang/AST/ASTConsumer.h" | 7 #include "clang/AST/ASTConsumer.h" |
8 #include "clang/Frontend/FrontendPluginRegistry.h" | 8 #include "clang/Frontend/FrontendPluginRegistry.h" |
9 | 9 |
10 #include "FindBadConstructsConsumer.h" | 10 #include "FindBadConstructsConsumer.h" |
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
42 } | 42 } |
43 | 43 |
44 bool FindBadConstructsAction::ParseArgs(const CompilerInstance& instance, | 44 bool FindBadConstructsAction::ParseArgs(const CompilerInstance& instance, |
45 const std::vector<std::string>& args) { | 45 const std::vector<std::string>& args) { |
46 bool parsed = true; | 46 bool parsed = true; |
47 | 47 |
48 for (size_t i = 0; i < args.size() && parsed; ++i) { | 48 for (size_t i = 0; i < args.size() && parsed; ++i) { |
49 if (args[i] == "check-base-classes") { | 49 if (args[i] == "check-base-classes") { |
50 // TODO(rsleevi): Remove this once http://crbug.com/123295 is fixed. | 50 // TODO(rsleevi): Remove this once http://crbug.com/123295 is fixed. |
51 options_.check_base_classes = true; | 51 options_.check_base_classes = true; |
52 } else if (args[i] == "enforce-blink") { | 52 } else if (args[i] == "enforce-overriding-blink") { |
53 options_.enforce_blink = true; | 53 options_.enforce_overriding_blink = true; |
| 54 } else if (args[i] == "enforce-in-thirdparty-webkit") { |
| 55 options_.enforce_in_thirdparty_webkit = true; |
| 56 options_.enforce_overriding_blink = true; |
54 } else if (args[i] == "check-enum-last-value") { | 57 } else if (args[i] == "check-enum-last-value") { |
55 // TODO(tsepez): Enable this by default once http://crbug.com/356815 | 58 // TODO(tsepez): Enable this by default once http://crbug.com/356815 |
56 // and http://crbug.com/356816 are fixed. | 59 // and http://crbug.com/356816 are fixed. |
57 options_.check_enum_last_value = true; | 60 options_.check_enum_last_value = true; |
58 } else if (args[i] == "with-ast-visitor") { | 61 } else if (args[i] == "with-ast-visitor") { |
59 options_.with_ast_visitor = true; | 62 options_.with_ast_visitor = true; |
60 } else if (args[i] == "check-templates") { | 63 } else if (args[i] == "check-templates") { |
61 options_.check_templates = true; | 64 options_.check_templates = true; |
62 } else if (args[i] == "warn-only") { | 65 } else if (args[i] == "warn-only") { |
63 options_.warn_only = true; | 66 options_.warn_only = true; |
64 } else { | 67 } else { |
65 parsed = false; | 68 parsed = false; |
66 llvm::errs() << "Unknown clang plugin argument: " << args[i] << "\n"; | 69 llvm::errs() << "Unknown clang plugin argument: " << args[i] << "\n"; |
67 } | 70 } |
68 } | 71 } |
69 | 72 |
70 return parsed; | 73 return parsed; |
71 } | 74 } |
72 | 75 |
73 } // namespace chrome_checker | 76 } // namespace chrome_checker |
74 | 77 |
75 static FrontendPluginRegistry::Add<chrome_checker::FindBadConstructsAction> X( | 78 static FrontendPluginRegistry::Add<chrome_checker::FindBadConstructsAction> X( |
76 "find-bad-constructs", | 79 "find-bad-constructs", |
77 "Finds bad C++ constructs"); | 80 "Finds bad C++ constructs"); |
OLD | NEW |