commit e2125fdeb1f72f5599853f82fc7ed2be9d5ee015
author Greg Kaiser <gkaiser@google.com> Tue Mar 26 11:58:53 2019 -0700
committer Greg Kaiser <gkaiser@google.com> Tue Mar 26 12:00:19 2019 -0700
tree 433739e6796aa6716c4f00088250b0430ad1e55d
parent d2c4eb0daeb3a668e6127a9f932bf313a0840be1

adb: Avoid extra std::string copies

When a function argument takes a std::string, it's inefficient to
pass std::string::c_str(), since that creates an additional copy.
So we change these calling sites.

Test: TreeHugger
Change-Id: I1c0fde7275eb3ebd3baf05ec62581b0243655608

adb/services.cpp

diff --git a/adb/services.cpp b/adb/services.cpp
index 80f9f79..cf346ba 100644
--- a/adb/services.cpp
+++ b/adb/services.cpp
@@ -188,7 +188,7 @@
     if (!strncmp(host.c_str(), "emu:", 4)) {
         connect_emulator(host.c_str() + 4, &response);
     } else {
-        connect_device(host.c_str(), &response);
+        connect_device(host, &response);
     }
 
     // Send response for emulator and device