xazax.hun added inline comments.

================
Comment at: clang-tidy/misc/SuspiciousMissingCommaCheck.cpp:75
@@ +74,3 @@
+  diag(ConcatenatedLiteral->getLocStart(),
+       "suspicious string literal, probably missing a comma");
+}
----------------
alexfh wrote:
> We need to add a recommendation on how to silence this warning (use 
> parentheses, for example). Fine for a follow up.
A possible way to silence this warning would be to implement a heuristic that 
is available in szdominik's implementation, i.e.: when the size of the array is 
explicitly given by the user and it matches the length of the initializer list 
do not warn.


http://reviews.llvm.org/D18457



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

Reply via email to