[PATCH] D22299: [include-fixer] Implement adding missing namespace qualifiers in vim integration.

[Haojian Wu via cfe-commits]

hokein marked an inline comment as done.

================
Comment at: include-fixer/tool/ClangIncludeFixer.cpp:252
@@ +251,3 @@
+    // Only accept an unique header.
+    bool IsUniqueHeader =
+        std::adjacent_find(HeaderInfos.begin(), HeaderInfos.end(),
----------------
You are right. `adjancent_find` is not correct algorithm here. What I want to do is to check all elements' headers in `HeaderInfos` are equal. Change to `std::equal`, the same below.

================
Comment at: include-fixer/tool/clang-include-fixer.py:119
@@ +118,3 @@
+  header_infos = include_fixer_context["HeaderInfos"]
+  # Reduplicate headers, so that the same header would not be suggested twice.
+  headers = list(set([header_info["Header"] for header_info in header_infos]))
----------------
Should be `Deduplicate`.


http://reviews.llvm.org/D22299