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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits