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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
