[clang-tools-extra] [clang-tools-extra] Use *Set::insert_range (NFC) (PR #133589)
Kazu Hirata via cfe-commits
cfe-commits at lists.llvm.org
Sat Mar 29 09:04:02 PDT 2025
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/133589
None
>From c6e9d5f59267dc346794e539fcbcd293fda84f53 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Fri, 28 Mar 2025 20:56:24 -0700
Subject: [PATCH] [clang-tools-extra] Use *Set::insert_range (NFC)
---
.../clang-tidy/bugprone/SignalHandlerCheck.cpp | 11 ++++-------
.../InconsistentDeclarationParameterNameCheck.cpp | 4 +---
clang-tools-extra/clangd/ConfigCompile.cpp | 3 +--
clang-tools-extra/clangd/index/dex/Dex.cpp | 3 +--
4 files changed, 7 insertions(+), 14 deletions(-)
diff --git a/clang-tools-extra/clang-tidy/bugprone/SignalHandlerCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/SignalHandlerCheck.cpp
index 902490f4d33c1..27045816a80d3 100644
--- a/clang-tools-extra/clang-tidy/bugprone/SignalHandlerCheck.cpp
+++ b/clang-tools-extra/clang-tidy/bugprone/SignalHandlerCheck.cpp
@@ -333,13 +333,10 @@ SignalHandlerCheck::SignalHandlerCheck(StringRef Name,
: ClangTidyCheck(Name, Context),
AsyncSafeFunctionSet(Options.get("AsyncSafeFunctionSet",
AsyncSafeFunctionSetKind::POSIX)) {
- if (AsyncSafeFunctionSet == AsyncSafeFunctionSetKind::Minimal) {
- for (StringRef v : MinimalConformingFunctions)
- ConformingFunctions.insert(v);
- } else {
- for (StringRef v : POSIXConformingFunctions)
- ConformingFunctions.insert(v);
- }
+ if (AsyncSafeFunctionSet == AsyncSafeFunctionSetKind::Minimal)
+ ConformingFunctions.insert_range(MinimalConformingFunctions);
+ else
+ ConformingFunctions.insert_range(POSIXConformingFunctions);
}
void SignalHandlerCheck::storeOptions(ClangTidyOptions::OptionMap &Opts) {
diff --git a/clang-tools-extra/clang-tidy/readability/InconsistentDeclarationParameterNameCheck.cpp b/clang-tools-extra/clang-tidy/readability/InconsistentDeclarationParameterNameCheck.cpp
index c41f81b0f0b58..a60a4faa32a16 100644
--- a/clang-tools-extra/clang-tidy/readability/InconsistentDeclarationParameterNameCheck.cpp
+++ b/clang-tools-extra/clang-tidy/readability/InconsistentDeclarationParameterNameCheck.cpp
@@ -343,9 +343,7 @@ void InconsistentDeclarationParameterNameCheck::check(
void InconsistentDeclarationParameterNameCheck::markRedeclarationsAsVisited(
const FunctionDecl *OriginalDeclaration) {
- for (const FunctionDecl *Redecl : OriginalDeclaration->redecls()) {
- VisitedDeclarations.insert(Redecl);
- }
+ VisitedDeclarations.insert_range(OriginalDeclaration->redecls());
}
} // namespace clang::tidy::readability
diff --git a/clang-tools-extra/clangd/ConfigCompile.cpp b/clang-tools-extra/clangd/ConfigCompile.cpp
index 21304a8c0fac7..3d7f792aa136b 100644
--- a/clang-tools-extra/clangd/ConfigCompile.cpp
+++ b/clang-tools-extra/clangd/ConfigCompile.cpp
@@ -439,8 +439,7 @@ struct FragmentCompiler {
[Normalized(std::move(Normalized))](const Params &, Config &C) {
if (C.Diagnostics.SuppressAll)
return;
- for (llvm::StringRef N : Normalized)
- C.Diagnostics.Suppress.insert(N);
+ C.Diagnostics.Suppress.insert_range(Normalized);
});
if (F.UnusedIncludes) {
diff --git a/clang-tools-extra/clangd/index/dex/Dex.cpp b/clang-tools-extra/clangd/index/dex/Dex.cpp
index 5643ba0c5e4ce..575a96a112979 100644
--- a/clang-tools-extra/clangd/index/dex/Dex.cpp
+++ b/clang-tools-extra/clangd/index/dex/Dex.cpp
@@ -181,8 +181,7 @@ std::unique_ptr<Iterator> Dex::createFileProximityIterator(
Sources[Path] = SourceParams();
auto PathURI = URI::create(Path).toString();
const auto PathProximityURIs = generateProximityURIs(PathURI.c_str());
- for (const auto &ProximityURI : PathProximityURIs)
- ParentURIs.insert(ProximityURI);
+ ParentURIs.insert_range(PathProximityURIs);
}
// Use SymbolRelevanceSignals for symbol relevance evaluation: use defaults
// for all parameters except for Proximity Path distance signal.
More information about the cfe-commits
mailing list