[PATCH] Refactor: Move remove-cstr-calls from a standalone executable to a clang-tidy check
Samuel Benzaquen
sbenza at google.com
Mon Feb 23 09:26:01 PST 2015
================
Comment at: clang-tidy/readability/RemoveCStrCall.cpp:37
@@ +36,3 @@
+bool needParensAfterUnaryOperator(const Expr &ExprNode) {
+ if (dyn_cast<clang::BinaryOperator>(&ExprNode) ||
+ dyn_cast<clang::ConditionalOperator>(&ExprNode)) {
----------------
Use isa<> instead
http://reviews.llvm.org/D7318
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list