[PATCH] D22299: [include-fixer] Implement adding missing namespace qualifiers in vim integration.
Benjamin Kramer via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 13 05:47:18 PDT 2016
bkramer 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(
----------------
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.
http://reviews.llvm.org/D22299
More information about the cfe-commits
mailing list