JonasToth marked 5 inline comments as done.
JonasToth added inline comments.


================
Comment at: clang-tidy/readability/IsolateDeclCheck.cpp:71
+  auto Diag =
+      diag(WholeDecl->getBeginLoc(), "make only one declaration per 
statement");
+
----------------
kbobyrev wrote:
> Maybe it's just me: this doesn't seem like a very clear diagnostic message, 
> probably needs better wording (can't think of anything in particular ATM, 
> though).
I updated the diagnostic to include the number of declared variables. I can 
reword again, for me its hard to think of something better.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D51949



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

Reply via email to