diff options
author | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2020-02-12 00:39:15 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2020-02-12 00:39:15 +0000 |
commit | 40ef6319ca56bda8f1b4719b4e8a1f8ae24e229f (patch) | |
tree | 343c79022fb83c0f6631b5db7eab2be3dc12dcfa /startop | |
parent | 2593e104d5cf829b2aa7a715530bebd63b3dc991 (diff) | |
parent | 7c0f0d3cecc59443d2aaab079e44e9630f732eab (diff) |
Merge "Fix clang-tidy performance-faster-string-find warnings" am: 57fb318ca3 am: 7c0f0d3cec
Change-Id: I181488a9d4c5aed3f62cf497ae6d092fcf27e7f6
Diffstat (limited to 'startop')
-rw-r--r-- | startop/view_compiler/dex_layout_compiler.cc | 4 | ||||
-rw-r--r-- | startop/view_compiler/util.cc | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/startop/view_compiler/dex_layout_compiler.cc b/startop/view_compiler/dex_layout_compiler.cc index cb820f8f20fb..bddb8aa6716a 100644 --- a/startop/view_compiler/dex_layout_compiler.cc +++ b/startop/view_compiler/dex_layout_compiler.cc @@ -118,7 +118,7 @@ namespace { std::string ResolveName(const std::string& name) { if (name == "View") return "android.view.View"; if (name == "ViewGroup") return "android.view.ViewGroup"; - if (name.find(".") == std::string::npos) { + if (name.find('.') == std::string::npos) { return StringPrintf("android.widget.%s", name.c_str()); } return name; @@ -205,4 +205,4 @@ void DexViewBuilder::PopViewStack() { view_stack_.pop_back(); } -} // namespace startop
\ No newline at end of file +} // namespace startop diff --git a/startop/view_compiler/util.cc b/startop/view_compiler/util.cc index a0637e6da32f..c34d7b059cfc 100644 --- a/startop/view_compiler/util.cc +++ b/startop/view_compiler/util.cc @@ -23,13 +23,13 @@ namespace util { // TODO: see if we can borrow this from somewhere else, like aapt2. string FindLayoutNameFromFilename(const string& filename) { - size_t start = filename.rfind("/"); + size_t start = filename.rfind('/'); if (start == string::npos) { start = 0; } else { start++; // advance past '/' character } - size_t end = filename.find(".", start); + size_t end = filename.find('.', start); return filename.substr(start, end - start); } |