=?utf-8?q?Björn_Schäpers?= <[email protected]>,
=?utf-8?q?Björn_Schäpers?= <[email protected]>,
=?utf-8?q?Björn_Schäpers?= <[email protected]>,
=?utf-8?q?Björn_Schäpers?= <[email protected]>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/[email protected]>
================
@@ -28,9 +27,12 @@ class SuspiciousIncludeCheck : public ClangTidyCheck {
SuspiciousIncludeCheck(StringRef Name, ClangTidyContext *Context);
void registerPPCallbacks(const SourceManager &SM, Preprocessor *PP,
Preprocessor *ModuleExpanderPP) override;
+ void storeOptions(ClangTidyOptions::OptionMap &Opts) override;
FileExtensionsSet HeaderFileExtensions;
FileExtensionsSet ImplementationFileExtensions;
+ std::optional<StringRef> IgnoredRegexString;
----------------
vbvictor wrote:
Could we drop `optional` and use plain `StringRef` here?
We use `IgnoredRegexString.value_or(StringRef{})` to construct regex so it
should be NFC change
https://github.com/llvm/llvm-project/pull/160958
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
