[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