Looks good. Nice!

================
Comment at: clang-tidy/google/AvoidCStyleCastsCheck.cpp:86
@@ +85,3 @@
+    diag_builder << ("Use " + CastType + ".").str();
+    if (ParenRange.getBegin().isFileID() && ParenRange.getEnd().isFileID()) {
+      diag_builder << FixItHint::CreateReplacement(
----------------
I'd probably prefer "if (!ParenRange ...) return".

http://reviews.llvm.org/D4478



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to