[PATCH] D7982: [clang-tidy] Add readability-duplicate-include check

[Richard via Phabricator via cfe-commits]

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