return 0;
}
+llvm::SmallVector<llvm::Regex>
+ReservedIdentifierCheck::parseAllowedIdentifiers() const {
+ llvm::SmallVector<llvm::Regex> AllowedIdentifiers;
+ AllowedIdentifiers.reserve(AllowedIdentifiersRaw.size());
+
+ for (const auto &Identifier : AllowedIdentifiersRaw) {
+ AllowedIdentifiers.emplace_back(Identifier.str());
+ if (!AllowedIdentifiers.back().isValid()) {
+ configurationDiag("Invalid allowed identifier regex '%0'") << Identifier;
+ AllowedIdentifiers.pop_back();
+ }
+ }
+
+ return AllowedIdentifiers;
+}
+
ReservedIdentifierCheck::ReservedIdentifierCheck(StringRef Name,
ClangTidyContext *Context)
: RenamerClangTidyCheck(Name, Context),
Invert(Options.get("Invert", false)),
- AllowedIdentifiers(utils::options::parseStringList(
- Options.get("AllowedIdentifiers", ""))) {}
+ AllowedIdentifiersRaw(utils::options::parseStringList(
+ Options.get("AllowedIdentifiers", ""))),
+ AllowedIdentifiers(parseAllowedIdentifiers()) {}
void ReservedIdentifierCheck::storeOptions(ClangTidyOptions::OptionMap &Opts) {
RenamerClangTidyCheck::storeOptions(Opts);
Options.store(Opts, "Invert", Invert);
Options.store(Opts, "AllowedIdentifiers",
- utils::options::serializeStringList(AllowedIdentifiers));
+ utils::options::serializeStringList(AllowedIdentifiersRaw));
}
static std::string collapseConsecutive(StringRef Str, char C) {
static std::optional<RenamerClangTidyCheck::FailureInfo>
getFailureInfoImpl(StringRef Name, bool IsInGlobalNamespace,
const LangOptions &LangOpts, bool Invert,
- ArrayRef<StringRef> AllowedIdentifiers) {
+ ArrayRef<llvm::Regex> AllowedIdentifiers) {
assert(!Name.empty());
- if (llvm::is_contained(AllowedIdentifiers, Name))
- return std::nullopt;
+ if (llvm::any_of(AllowedIdentifiers, [&](const llvm::Regex &Regex) {
+ return Regex.match(Name);
+ })) {
+ return std::nullopt;
+ }
// TODO: Check for names identical to language keywords, and other names
// specifically reserved by language standards, e.g. C++ 'zombie names' and C
// future library directions
/// http://clang.llvm.org/extra/clang-tidy/checks/bugprone/reserved-identifier.html
class ReservedIdentifierCheck final : public RenamerClangTidyCheck {
const bool Invert;
- const std::vector<StringRef> AllowedIdentifiers;
+ const std::vector<StringRef> AllowedIdentifiersRaw;
+ const llvm::SmallVector<llvm::Regex> AllowedIdentifiers;
public:
ReservedIdentifierCheck(StringRef Name, ClangTidyContext *Context);
const SourceManager &SM) const override;
DiagInfo getDiagInfo(const NamingCheckId &ID,
const NamingCheckFailure &Failure) const override;
+ llvm::SmallVector<llvm::Regex> parseAllowedIdentifiers() const;
};
} // namespace clang::tidy::bugprone
- Improved the performance of the :doc:`bugprone-reserved-identifier
<clang-tidy/checks/bugprone/reserved-identifier>` check through optimizations.
+- Improved the :doc:`bugprone-reserved-identifier
+ <clang-tidy/checks/bugprone/reserved-identifier>` check by enhancing the
+ `AllowedIdentifiers` option to support regular expressions.
+
- Deprecated check-local options `HeaderFileExtensions` and `ImplementationFileExtensions`
in :doc:`bugprone-suspicious-include
<clang-tidy/checks/bugprone/suspicious-include>` check.
// RUN: %check_clang_tidy %s bugprone-reserved-identifier %t -- \
// RUN: -config='{CheckOptions: [ \
// RUN: {key: bugprone-reserved-identifier.Invert, value: true}, \
-// RUN: {key: bugprone-reserved-identifier.AllowedIdentifiers, value: std;reference_wrapper;ref;cref;type;get}, \
+// RUN: {key: bugprone-reserved-identifier.AllowedIdentifiers, value: "std;reference_wrapper;^c?ref;type;get"}, \
// RUN: ]}' -- \
// RUN: -I%S/Inputs/reserved-identifier \
// RUN: -isystem %S/Inputs/reserved-identifier/system