[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