[PATCH] D13133: Remove dangling parenthesis.

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 24 07:39:19 PDT 2015


klimek added inline comments.

================
Comment at: clang-tidy/modernize/LoopConvertCheck.cpp:482
@@ +481,3 @@
+        auto Parents = Context->getParents(*Usage.Expression);
+        if (Parents.size() == 1) {
+          if (const auto *Paren = Parents[0].get<ParenExpr>())
----------------
Perhaps add a comment (or correct me if I'm wrong ;)
// Usage.Expression contains the DeclRefExpr from the point of usage.
// Parens around a simple DeclRefExpr can always be removed.

(can Usage.Expression be something more complex?)


http://reviews.llvm.org/D13133





More information about the cfe-commits mailing list