commit a89cdccb7329b8b883abef811ff3938c291068a2
author Josh Gao <jmgao@google.com> Mon Feb 25 13:02:43 2019 -0800
committer Josh Gao <jmgao@google.com> Mon Feb 25 13:59:00 2019 -0800
tree 606ac05eda5b64ad18aec50427a86a5baf325947
parent 6b4b800cefbe1ba4ecf5c89eddc216b134e3fac5

adbd: switch abb to ConsumePrefix.

Test: mma
Change-Id: I57d1a30a526c97c5b5a2718740b76220da6eea39

adb/daemon/abb.cpp

diff --git a/adb/daemon/abb.cpp b/adb/daemon/abb.cpp
index 4ffa6bb..eeac41a 100644
--- a/adb/daemon/abb.cpp
+++ b/adb/daemon/abb.cpp
@@ -14,15 +14,15 @@
  * limitations under the License.
  */
 
-#include "adb.h"
-#include "adb_io.h"
-#include "shell_service.h"
-
-#include "cmd.h"
-
 #include <sys/wait.h>
 
 #include <android-base/cmsg.h>
+#include <cmd.h>
+
+#include "adb.h"
+#include "adb_io.h"
+#include "adb_utils.h"
+#include "shell_service.h"
 
 namespace {
 
@@ -87,11 +87,9 @@
 
         std::string_view name = data;
         auto protocol = SubprocessProtocol::kShell;
-        if (name.starts_with("abb:")) {
-            name.remove_prefix(strlen("abb:"));
+        if (ConsumePrefix(&name, "abb:")) {
             protocol = SubprocessProtocol::kShell;
-        } else if (name.starts_with("abb_exec:")) {
-            name.remove_prefix(strlen("abb_exec:"));
+        } else if (ConsumePrefix(&name, "abb_exec:")) {
             protocol = SubprocessProtocol::kNone;
         } else {
             LOG(FATAL) << "Unknown command prefix for abb: " << data;