hokein added a comment.
> You might want to commit this at some point ;]
I somewhat forgot this minor patch.
================
Comment at: clang-tidy/google/AvoidCStyleCastsCheck.cpp:139
")");
+ ReplaceRange = CharSourceRange::getCharRange(CastExpr->getLParenLoc(),
+ SubExpr->getLocStart());
----------------
alexfh wrote:
> Why is this only done on non-parenthesized expressions? Will `(int) (b)`
> turn into `static_cast<int> (b)` as well?
Good point, I have updated the patch to cover more cases. Please take a look
again.
https://reviews.llvm.org/D31700
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
