[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