diff options
author | Eric Christopher <echristo@gmail.com> | 2020-06-19 23:01:42 -0700 |
---|---|---|
committer | Eric Christopher <echristo@gmail.com> | 2020-06-19 23:01:42 -0700 |
commit | 25ed42f05d07d3ad85c58ee001752338211c911c (patch) | |
tree | b5167645cda61542fd772a39f3e462ebc9cc747c /clang-tools-extra/clang-change-namespace | |
parent | 8027f04a6d438b83b75856b03278122f14d6b1d3 (diff) | |
download | llvm-25ed42f05d07d3ad85c58ee001752338211c911c.tar.gz |
[clang-change-namespace] As part of using inclusive language
within the llvm project, migrate away from the use of blacklist
and whitelist.
Diffstat (limited to 'clang-tools-extra/clang-change-namespace')
3 files changed, 17 insertions, 17 deletions
diff --git a/clang-tools-extra/clang-change-namespace/ChangeNamespace.cpp b/clang-tools-extra/clang-change-namespace/ChangeNamespace.cpp index 5f4a889cab9e..e2a70db4102b 100644 --- a/clang-tools-extra/clang-change-namespace/ChangeNamespace.cpp +++ b/clang-tools-extra/clang-change-namespace/ChangeNamespace.cpp @@ -347,7 +347,7 @@ bool isTemplateParameter(TypeLoc Type) { ChangeNamespaceTool::ChangeNamespaceTool( llvm::StringRef OldNs, llvm::StringRef NewNs, llvm::StringRef FilePattern, - llvm::ArrayRef<std::string> WhiteListedSymbolPatterns, + llvm::ArrayRef<std::string> AllowedSymbolPatterns, std::map<std::string, tooling::Replacements> *FileToReplacements, llvm::StringRef FallbackStyle) : FallbackStyle(FallbackStyle), FileToReplacements(*FileToReplacements), @@ -365,8 +365,8 @@ ChangeNamespaceTool::ChangeNamespaceTool( DiffOldNamespace = joinNamespaces(OldNsSplitted); DiffNewNamespace = joinNamespaces(NewNsSplitted); - for (const auto &Pattern : WhiteListedSymbolPatterns) - WhiteListedSymbolRegexes.emplace_back(Pattern); + for (const auto &Pattern : AllowedSymbolPatterns) + AllowedSymbolRegexes.emplace_back(Pattern); } void ChangeNamespaceTool::registerMatchers(ast_matchers::MatchFinder *Finder) { @@ -800,7 +800,7 @@ void ChangeNamespaceTool::replaceQualifiedSymbolInDeclContext( Result.SourceManager->getSpellingLoc(End)), *Result.SourceManager, Result.Context->getLangOpts()); std::string FromDeclName = FromDecl->getQualifiedNameAsString(); - for (llvm::Regex &RE : WhiteListedSymbolRegexes) + for (llvm::Regex &RE : AllowedSymbolRegexes) if (RE.match(FromDeclName)) return; std::string ReplaceName = diff --git a/clang-tools-extra/clang-change-namespace/ChangeNamespace.h b/clang-tools-extra/clang-change-namespace/ChangeNamespace.h index 147675911941..d35119b70a69 100644 --- a/clang-tools-extra/clang-change-namespace/ChangeNamespace.h +++ b/clang-tools-extra/clang-change-namespace/ChangeNamespace.h @@ -49,7 +49,7 @@ public: // files matching `FilePattern`. ChangeNamespaceTool( llvm::StringRef OldNs, llvm::StringRef NewNs, llvm::StringRef FilePattern, - llvm::ArrayRef<std::string> WhiteListedSymbolPatterns, + llvm::ArrayRef<std::string> AllowedSymbolPatterns, std::map<std::string, tooling::Replacements> *FileToReplacements, llvm::StringRef FallbackStyle = "LLVM"); @@ -166,7 +166,7 @@ private: llvm::SmallPtrSet<const clang::DeclRefExpr*, 16> ProcessedFuncRefs; // Patterns of symbol names whose references are not expected to be updated // when changing namespaces around them. - std::vector<llvm::Regex> WhiteListedSymbolRegexes; + std::vector<llvm::Regex> AllowedSymbolRegexes; }; } // namespace change_namespace diff --git a/clang-tools-extra/clang-change-namespace/tool/ClangChangeNamespace.cpp b/clang-tools-extra/clang-change-namespace/tool/ClangChangeNamespace.cpp index d54fe7347cf3..61754fe41bc1 100644 --- a/clang-tools-extra/clang-change-namespace/tool/ClangChangeNamespace.cpp +++ b/clang-tools-extra/clang-change-namespace/tool/ClangChangeNamespace.cpp @@ -72,20 +72,20 @@ cl::opt<std::string> Style("style", cl::desc("The style name used for reformatting."), cl::init("LLVM"), cl::cat(ChangeNamespaceCategory)); -cl::opt<std::string> WhiteListFile( - "whitelist_file", +cl::opt<std::string> AllowedFile( + "allowed_file", cl::desc("A file containing regexes of symbol names that are not expected " "to be updated when changing namespaces around them."), cl::init(""), cl::cat(ChangeNamespaceCategory)); -llvm::ErrorOr<std::vector<std::string>> GetWhiteListedSymbolPatterns() { +llvm::ErrorOr<std::vector<std::string>> GetAllowedSymbolPatterns() { std::vector<std::string> Patterns; - if (WhiteListFile.empty()) + if (AllowedFile.empty()) return Patterns; llvm::SmallVector<StringRef, 8> Lines; llvm::ErrorOr<std::unique_ptr<llvm::MemoryBuffer>> File = - llvm::MemoryBuffer::getFile(WhiteListFile); + llvm::MemoryBuffer::getFile(AllowedFile); if (!File) return File.getError(); llvm::StringRef Content = File.get()->getBuffer(); @@ -103,15 +103,15 @@ int main(int argc, const char **argv) { ChangeNamespaceCategory); const auto &Files = OptionsParser.getSourcePathList(); tooling::RefactoringTool Tool(OptionsParser.getCompilations(), Files); - llvm::ErrorOr<std::vector<std::string>> WhiteListPatterns = - GetWhiteListedSymbolPatterns(); - if (!WhiteListPatterns) { - llvm::errs() << "Failed to open whitelist file " << WhiteListFile << ". " - << WhiteListPatterns.getError().message() << "\n"; + llvm::ErrorOr<std::vector<std::string>> AllowedPatterns = + GetAllowedSymbolPatterns(); + if (!AllowedPatterns) { + llvm::errs() << "Failed to open Allowed file " << AllowedFile << ". " + << AllowedPatterns.getError().message() << "\n"; return 1; } change_namespace::ChangeNamespaceTool NamespaceTool( - OldNamespace, NewNamespace, FilePattern, *WhiteListPatterns, + OldNamespace, NewNamespace, FilePattern, *AllowedPatterns, &Tool.getReplacements(), Style); ast_matchers::MatchFinder Finder; NamespaceTool.registerMatchers(&Finder); |