From c4dfa03f9f44fa183daabdd4e6d760a432ef6531 Mon Sep 17 00:00:00 2001 From: Kazu Hirata Date: Wed, 6 Nov 2024 08:35:44 -0800 Subject: [PATCH] [clang-tidy] Call StringMap::find without constructing std::string (NFC) (#115114) StringMap::find takes StringRef, so we don't need to allocate temporary instances of std::string. --- clang-tools-extra/clang-tidy/abseil/DurationRewriter.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/clang-tools-extra/clang-tidy/abseil/DurationRewriter.cpp b/clang-tools-extra/clang-tidy/abseil/DurationRewriter.cpp index 3d807c1d0f7a..4cdbbd43c143 100644 --- a/clang-tools-extra/clang-tidy/abseil/DurationRewriter.cpp +++ b/clang-tools-extra/clang-tidy/abseil/DurationRewriter.cpp @@ -244,7 +244,7 @@ std::optional getScaleForDurationInverse(llvm::StringRef Name) { {"ToDoubleNanoseconds", DurationScale::Nanoseconds}, {"ToInt64Nanoseconds", DurationScale::Nanoseconds}}); - auto ScaleIter = ScaleMap.find(std::string(Name)); + auto ScaleIter = ScaleMap.find(Name); if (ScaleIter == ScaleMap.end()) return std::nullopt; @@ -260,7 +260,7 @@ std::optional getScaleForTimeInverse(llvm::StringRef Name) { {"ToUnixMicros", DurationScale::Microseconds}, {"ToUnixNanos", DurationScale::Nanoseconds}}); - auto ScaleIter = ScaleMap.find(std::string(Name)); + auto ScaleIter = ScaleMap.find(Name); if (ScaleIter == ScaleMap.end()) return std::nullopt;