Skip to content

Commit 4bac5f8

Browse files
Apply fixes from performance-faster-string-find (NFC)
1 parent 1ca496b commit 4bac5f8

File tree

4 files changed

+10
-10
lines changed

4 files changed

+10
-10
lines changed

clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -337,10 +337,10 @@ std::string IdentifierNamingCheck::HungarianNotation::getDeclTypeName(
337337
Type.replace(Pos, Kw.size(), "");
338338
}
339339
}
340-
TypeName = Type.erase(0, Type.find_first_not_of(" "));
340+
TypeName = Type.erase(0, Type.find_first_not_of(' '));
341341

342342
// Remove template parameters
343-
const size_t Pos = Type.find("<");
343+
const size_t Pos = Type.find('<');
344344
if (Pos != std::string::npos) {
345345
TypeName = Type.erase(Pos, Type.size() - Pos);
346346
}
@@ -377,14 +377,14 @@ std::string IdentifierNamingCheck::HungarianNotation::getDeclTypeName(
377377
}
378378
}
379379

380-
TypeName = Type.erase(0, Type.find_first_not_of(" "));
380+
TypeName = Type.erase(0, Type.find_first_not_of(' '));
381381
if (!RedundantRemoved) {
382-
std::size_t FoundSpace = Type.find(" ");
382+
std::size_t FoundSpace = Type.find(' ');
383383
if (FoundSpace != std::string::npos)
384384
Type = Type.substr(0, FoundSpace);
385385
}
386386

387-
TypeName = Type.erase(0, Type.find_first_not_of(" "));
387+
TypeName = Type.erase(0, Type.find_first_not_of(' '));
388388

389389
QualType QT = VD->getType();
390390
if (!QT.isNull() && QT->isArrayType())
@@ -586,7 +586,7 @@ std::string IdentifierNamingCheck::HungarianNotation::getDataTypePrefix(
586586
if (PrefixStr.empty())
587587
PrefixStr = HNOption.DerivedType.lookup("Array");
588588
} else if (QT->isReferenceType()) {
589-
size_t Pos = ModifiedTypeName.find_last_of("&");
589+
size_t Pos = ModifiedTypeName.find_last_of('&');
590590
if (Pos != std::string::npos)
591591
ModifiedTypeName = ModifiedTypeName.substr(0, Pos);
592592
}
@@ -653,7 +653,7 @@ std::string IdentifierNamingCheck::HungarianNotation::getEnumPrefix(
653653
std::string Name = ED->getName().str();
654654
if (std::string::npos != Name.find("enum")) {
655655
Name = Name.substr(strlen("enum"), Name.length() - strlen("enum"));
656-
Name = Name.erase(0, Name.find_first_not_of(" "));
656+
Name = Name.erase(0, Name.find_first_not_of(' '));
657657
}
658658

659659
static llvm::Regex Splitter(

flang/lib/Semantics/check-io.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ void IoChecker::Enter(const parser::ConnectSpec &spec) {
101101
// Ignore trailing spaces (12.5.6.2 p1) and convert to upper case
102102
static std::string Normalize(const std::string &value) {
103103
auto upper{parser::ToUpperCaseLetters(value)};
104-
std::size_t lastNonBlank{upper.find_last_not_of(" ")};
104+
std::size_t lastNonBlank{upper.find_last_not_of(' ')};
105105
upper.resize(lastNonBlank == std::string::npos ? 0 : lastNonBlank + 1);
106106
return upper;
107107
}

lldb/source/Plugins/Process/Linux/IntelPTCollector.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ static std::optional<int> GetCGroupFileDescriptor(lldb::pid_t pid) {
8181
if (line.find("0:") != 0)
8282
continue;
8383

84-
std::string slice = line.substr(line.find_first_of("/"));
84+
std::string slice = line.substr(line.find_first_of('/'));
8585
if (slice.empty())
8686
return std::nullopt;
8787
std::string cgroup_file = formatv("/sys/fs/cgroup/{0}", slice);

llvm/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1256,7 +1256,7 @@ void DataFlowSanitizer::addGlobalNameSuffix(GlobalValue *GV) {
12561256
size_t Pos = Asm.find(SearchStr);
12571257
if (Pos != std::string::npos) {
12581258
Asm.replace(Pos, SearchStr.size(), ".symver " + GVName + Suffix + ",");
1259-
Pos = Asm.find("@");
1259+
Pos = Asm.find('@');
12601260

12611261
if (Pos == std::string::npos)
12621262
report_fatal_error(Twine("unsupported .symver: ", Asm));

0 commit comments

Comments
 (0)