sbenza accepted this revision.
sbenza added a comment.
This revision is now accepted and ready to land.
See comment regarding //CHECKs
================
Comment at: test/clang-tidy/cert-variadic-function-def.cpp:4
@@ +3,3 @@
+// Variadic function definitions are diagnosed.
+// CHECK-MESSAGES: :[[@LINE+1]]:6: warning: do not define a C-style variadic
function; consider using a function parameter pack or currying instead
[cert-dcl50-cpp]
+void f1(int, ...) {}
----------------
All other tests I've read/written put the //CHECK after the matched lines.
We should be consistent with that.
http://reviews.llvm.org/D13446
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
