[PATCH] D19534: [clang-tidy] new google-default-arguments check
Haojian Wu via cfe-commits
cfe-commits at lists.llvm.org
Fri May 6 02:25:59 PDT 2016
hokein accepted this revision.
hokein added a comment.
This revision is now accepted and ready to land.
LGTM with one nit.
================
Comment at: test/clang-tidy/google-default-arguments.cpp:10
@@ +9,3 @@
+ void f(int I, int J = 5);
+ // CHECK-MESSAGES: :[[@LINE-1]]:8: warning: default arguments on virtual or override methods are prohibited [google-default-arguments]
+};
----------------
You can remove the `[google-default-arguments]` in this line and `line 15`. Usually we only keep it on the first warning message.
http://reviews.llvm.org/D19534
More information about the cfe-commits
mailing list