commit 300176a2d8c2afb99cccfa26ac0e5a66b5b7d2cd
author Josh Gao <jmgao@google.com> Wed Feb 28 14:44:23 2018 -0800
committer Josh Gao <jmgao@google.com> Mon Mar 05 13:00:28 2018 -0800
tree 45bc4989824ee26b6d59a64542971749cc5cfe37
parent 5fee1bb8f189ca84ed2d08c75bc13cb5bec2f4f7

adb: remove meaningless const.

Returning a const value is pointless.

Test: mma
Change-Id: Ia578d75345361d53e4a29bb9724bfcc161b8d8be

adb/transport.cpp

diff --git a/adb/transport.cpp b/adb/transport.cpp
index 1ccff92..3329f0f 100644
--- a/adb/transport.cpp
+++ b/adb/transport.cpp
@@ -709,7 +709,7 @@
     connection_state_ = state;
 }
 
-const std::string atransport::connection_state_name() const {
+std::string atransport::connection_state_name() const {
     ConnectionState state = GetConnectionState();
     switch (state) {
         case kCsOffline:

adb/transport.h

diff --git a/adb/transport.h b/adb/transport.h
index a492f00..8b71e34 100644
--- a/adb/transport.h
+++ b/adb/transport.h
@@ -204,8 +204,8 @@
     char token[TOKEN_SIZE] = {};
     size_t failed_auth_attempts = 0;
 
-    const std::string serial_name() const { return serial ? serial : "<unknown>"; }
-    const std::string connection_state_name() const;
+    std::string serial_name() const { return serial ? serial : "<unknown>"; }
+    std::string connection_state_name() const;
 
     void update_version(int version, size_t payload);
     int get_protocol_version() const;