klimek added inline comments.
================
Comment at: clang-tidy/modernize/LoopConvertCheck.cpp:371-374
@@ -370,1 +370,6 @@
+/// \brief Returns false when it can be guaranteed that no element of the
+/// container we are iterating over is going to be modified due to this
+/// expression.
+static bool canBeModified(ASTContext *Context, const Expr *E) {
+ auto Parents = Context->getParents(*E);
----------------
Not sure that is clearer. The comment should rather say what "E" is regarding
the container.
http://reviews.llvm.org/D14198
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
