================
@@ -62,10 +61,32 @@ class DuplicateIncludeCallbacks : public PPCallbacks {
SmallVector<FileList> Files;
DuplicateIncludeCheck &Check;
const SourceManager &SM;
+ SmallVector<llvm::Regex> AllowedRegexes;
};
} // namespace
+DuplicateIncludeCheck::DuplicateIncludeCheck(StringRef Name,
+ ClangTidyContext *Context)
+ : ClangTidyCheck(Name, Context),
+ IgnoredFilesList(
+ llvm::ArrayRef<llvm::StringRef>(utils::options::parseStringList(
----------------
vbvictor wrote:
`llvm::ArrayRef<llvm::StringRef>()` looks odd, I think we usually use
`utils::options::parseStringList` without any casting. I guess that `parseList`
returns `std::vector` thus we need it.
Despite @zwuis comment, I think we should use `std::vector` in options and
later create an overload for `utils::options::parseStringList` to return
`llvm::SmallVector`.
https://github.com/llvm/llvm-project/pull/168196
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
