JonasToth added a comment.

Updated



================
Comment at: clang-tidy/cppcoreguidelines/MacroUsageCheck.cpp:78
+        "function-like macro used; consider a 'constexpr' template function";
+  if (Info->isVariadic())
+    DiagnosticMessage = "variadic macro used; consider using a 'constexpr' "
----------------
aaron.ballman wrote:
> Aren't all vararg macros also function-like macros, so this will trigger two 
> diagnostics for all variadic macro definitions?
`MacroInfo` contains bits about function-like and beeing variadic separatly 
(gnu variadic as well, but thats included in `isVariadic`). There are no double 
warnings created.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D41648



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to