summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTreehugger Robot <treehugger-gerrit@google.com>2020-02-12 00:01:49 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2020-02-12 00:01:49 +0000
commit2658a27ac27defed751ebffed305cdfef0cffdb3 (patch)
tree143b5e17a93e3108ff2af86d854346024278d435
parent49220d1ad49bf4f0a141b4a2503abe8b661c7813 (diff)
parent3c5692007b8f7d0ca15fb6547871487a2fe73953 (diff)
downloadextras-2658a27ac27defed751ebffed305cdfef0cffdb3.tar.gz
Merge "Fix clang-tidy performance-faster-string-find warnings"
-rw-r--r--simpleperf/event_type.cpp4
-rw-r--r--simpleperf/event_type.h2
2 files changed, 3 insertions, 3 deletions
diff --git a/simpleperf/event_type.cpp b/simpleperf/event_type.cpp
index c9eaa75b..71c3d9f1 100644
--- a/simpleperf/event_type.cpp
+++ b/simpleperf/event_type.cpp
@@ -161,7 +161,7 @@ static uint64_t MakeEventConfig(const std::string& event_str, std::vector<EventF
// # cat armv8_pmuv3/events/cpu_cycles
// event=0x011
for (auto& s : android::base::Split(event_str, ",")) {
- auto pos = s.find("=");
+ auto pos = s.find('=');
if (pos == std::string::npos)
continue;
@@ -229,7 +229,7 @@ std::vector<int> EventType::GetPmuCpumask() {
if (!IsPmuEvent())
return empty_result;
- std::string pmu = name.substr(0, name.find("/"));
+ std::string pmu = name.substr(0, name.find('/'));
std::string cpumask_path = "/sys/bus/event_source/devices/" + pmu + "/cpumask";
std::string cpumask_content;
if (!android::base::ReadFileToString(cpumask_path, &cpumask_content)) {
diff --git a/simpleperf/event_type.h b/simpleperf/event_type.h
index d4ffecb7..359438ec 100644
--- a/simpleperf/event_type.h
+++ b/simpleperf/event_type.h
@@ -44,7 +44,7 @@ struct EventType {
}
bool IsPmuEvent() const {
- return name.find("/") != std::string::npos;
+ return name.find('/') != std::string::npos;
}
std::vector<int> GetPmuCpumask();