angelgarcia added inline comments.
================
Comment at: clang-tidy/modernize/LoopConvertCheck.cpp:659-664
@@ -646,7 +658,8 @@
+ BeginPointeeType.isTriviallyCopyableType(*Context);
} else {
// Check for qualified types to avoid conversions from non-const to const
// iterator types.
if (!Context->hasSameType(CanonicalInitVarType, CanonicalBeginType))
return;
}
----------------
Yes, but the arrays never return rvalues, and the iterator and pseudoarrays are
"handled" in different parts in the code. I think I updated both of them, and I
ensured that there was a test for each.
http://reviews.llvm.org/D12675
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
