[PATCH] D13787: [clang-tidy] add check cppcoreguidelines-pro-type-vararg-use
Samuel Benzaquen via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 20 13:01:31 PDT 2015
sbenza added a comment.
Minor comments only.
================
Comment at: clang-tidy/cppcoreguidelines/ProTypeVarargCheck.cpp:29
@@ +28,3 @@
+ callExpr(callee(functionDecl(isVariadic(),
+ unless(hasName("__builtin_va_start")))))
+ .bind("callvararg"),
----------------
No need to mention __builtin_va_start anymore.
================
Comment at: test/clang-tidy/cppcoreguidelines-pro-type-vararg.cpp:31
@@ +30,3 @@
+template <typename T>
+void CallFooIfAvailableImpl(T& t, decltype(t.foo())*) {
+ t->foo();
----------------
nit: t.foo() here vs t->foo() in the next line.
http://reviews.llvm.org/D13787
More information about the cfe-commits
mailing list