commit ce5f7d6423211cdaa4fe90133a9abef3aea35945
author Greg Kaiser <gkaiser@google.com> Fri Mar 27 06:10:11 2020 -0700
committer Greg Kaiser <gkaiser@google.com> Tue Apr 21 20:34:44 2020 -0700
tree 71624f00926aa43dd9d1cdf15ad09e798f827bc6
parent 9286d011702d880c4fbf9e80d328b9d19994d0a1

adb: Avoid extra string construction

Bug: 150827486
Test: TreeHugger
Change-Id: Ie03062a17b98866f1bc419a7e509461cf4e6bb80
Merged-In: Ie03062a17b98866f1bc419a7e509461cf4e6bb80

adb/client/file_sync_client.cpp

diff --git a/adb/client/file_sync_client.cpp b/adb/client/file_sync_client.cpp
index 190c235..e686973 100644
--- a/adb/client/file_sync_client.cpp
+++ b/adb/client/file_sync_client.cpp
@@ -617,7 +617,7 @@
         }
 
         std::string path_and_mode = android::base::StringPrintf("%s,%d", path.c_str(), mode);
-        if (!SendRequest(ID_SEND_V1, path_and_mode.c_str())) {
+        if (!SendRequest(ID_SEND_V1, path_and_mode)) {
             Error("failed to send ID_SEND_V1 message '%s': %s", path_and_mode.c_str(),
                   strerror(errno));
             return false;