[clang-tools-extra] clang tidy: add header ignore option to include duplicate files (PR #167046)

[Yanzuo Liu via cfe-commits]

================
@@ -19,11 +20,15 @@ namespace clang::tidy::readability {
 /// directives between them are analyzed.
 class DuplicateIncludeCheck : public ClangTidyCheck {
 public:
-  DuplicateIncludeCheck(StringRef Name, ClangTidyContext *Context)
-      : ClangTidyCheck(Name, Context) {}
+  DuplicateIncludeCheck(StringRef Name, ClangTidyContext *Context);
 
   void registerPPCallbacks(const SourceManager &SM, Preprocessor *PP,
                            Preprocessor *ModuleExpanderPP) override;
+
+  void storeOptions(ClangTidyOptions::OptionMap &Opts) override;
+
+private:
+  std::vector<std::string> AllowedDuplicateIncludes;
----------------
zwuis wrote:

`std::vector` and `std::string` are not preferred here. See the return type of `Options.get` and <https://llvm.org/docs/CodingStandards.html#c-standard-library>.

https://github.com/llvm/llvm-project/pull/167046