djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land.
Looks good. ================ Comment at: lib/Tooling/Core/Replacement.cpp:407 @@ -409,3 +406,3 @@ bool Result = true; - for (Replacements::const_iterator I = Replaces.begin(), - E = Replaces.end(); + for (Replacements::const_reverse_iterator I = Replaces.rbegin(), + E = Replaces.rend(); ---------------- Maybe use auto? https://reviews.llvm.org/D24663 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
