LegalizeAdulthood added a comment. In D7982#3216332 <https://reviews.llvm.org/D7982#3216332>, @Eugene.Zelenko wrote:
> Functionality-wise this check is superseded by Include What You Use > <https://github.com/include-what-you-use/include-what-you-use>. That's not part of clang-tidy and has it's own set of problems (false positives) when you run it. ================ Comment at: clang-tools-extra/docs/clang-tidy/checks/readability-duplicate-include.rst:6 + +Looks for duplicate includes and removes them. The check maintains a list of +included files and looks for duplicates. If a macro is defined or undefined ---------------- Eugene.Zelenko wrote: > Please fix double spaces. Same below. Please point me to a LLVM coding standards section that requires this. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D7982/new/ https://reviews.llvm.org/D7982 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
