alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.

Sorry, missed your patch somehow. LG with one nit.



================
Comment at: clang-tidy/google/AvoidCStyleCastsCheck.cpp:139
           ")");
+      ReplaceRange = CharSourceRange::getCharRange(CastExpr->getLParenLoc(),
+                                                   SubExpr->getLocStart());
----------------
Why is this only done on non-parenthesized expressions? Will `(int)  (b)` turn 
into `static_cast<int>  (b)` as well?


https://reviews.llvm.org/D31700



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

Reply via email to