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

Unified Diff: tools/clang/plugins/tests/test.sh

Issue 1493813003: Convert the no-inline-virtuals rule into a constructors rule. Base URL: https://chromium.googlesource.com/chromium/src.git@lkcr
Patch Set: Rebase onto https://codereview.chromium.org/1504033010 Created 5 years 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
Index: tools/clang/plugins/tests/test.sh
diff --git a/tools/clang/plugins/tests/test.sh b/tools/clang/plugins/tests/test.sh
index b096d1cdbdf58dddd1597bcbee487de125ea7f5a..ece9cb36bdef247ddae2770b3daea086fc4bd560 100755
--- a/tools/clang/plugins/tests/test.sh
+++ b/tools/clang/plugins/tests/test.sh
@@ -43,6 +43,8 @@ do_testcase() {
-Xclang with-ast-visitor"
flags="${flags} -Xclang -plugin-arg-find-bad-constructs \
-Xclang follow-macro-expansion"
+ flags="${flags} -Xclang -plugin-arg-find-bad-constructs \
dcheng 2016/03/24 17:37:25 You might need to rebase this, since I ported the
+ -Xclang treat-virtuals-as-complexity"
local output="$("${CLANG_PATH}" -fsyntax-only -Wno-c++11-extensions \
-Wno-inconsistent-missing-override \

Powered by Google App Engine
This is Rietveld 408576698