hokein added inline comments. ================ Comment at: include-fixer/tool/ClangIncludeFixer.cpp:252 @@ +251,3 @@ + assert(!HeaderInfos.empty()); + // Only accept an unique header. + bool IsUniqueHeader = std::equal( ---------------- bkramer wrote: > In that case adjacent_find was the right choice, but std::equal also works, I > read this wrong. There are multiple ways of doing this using standard > algorithms and none of them is really obvious. Please add a comment here that > you're checking that all elements are equal. Done. To me, `equal` is more obvious than `adjacent_find`.
http://reviews.llvm.org/D22299 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits