commit 0d9529861d9050dcef94fab7e7b04e127dfb177b
author Myles Watson <mylesgw@google.com> Fri May 08 10:53:49 2020 -0700
committer Myles Watson <mylesgw@google.com> Sat May 09 00:32:21 2020 +0000
tree e99b2d89e9824ab0eb451581ec04bc9366a3d3d5
parent 4a83dc0fd59ca637435c4cac7c836967ee979e93

ClassicConnection: Clarify packet-drop message

Tag: #gd-refactor
Bug: 145832107
Test: atest bluetooth_test_gd
Test: gd/cert/run --host
Change-Id: I7cbffc1a9d928ec6dbb9e494a726a8a45a937d39

system/gd/hci/acl_manager/assembler.h

diff --git a/system/gd/hci/acl_manager/assembler.h b/system/gd/hci/acl_manager/assembler.h
index e0a67de..186a5c0 100644
--- a/system/gd/hci/acl_manager/assembler.h
+++ b/system/gd/hci/acl_manager/assembler.h
@@ -73,7 +73,6 @@
   }
 
   void on_incoming_packet(AclPacketView packet) {
-    // TODO: What happens if the connection is stalled and fills up?
     PacketView<kLittleEndian> payload = packet.GetPayload();
     auto payload_size = payload.size();
     auto packet_boundary_flag = packet.GetPacketBoundaryFlag();
@@ -108,7 +107,7 @@
       }
     }
     if (incoming_queue_.size() > kMaxQueuedPacketsPerConnection) {
-      LOG_ERROR("Dropping packet due to congestion from remote:%s", address_with_type_.ToString().c_str());
+      LOG_ERROR("Dropping packet from %s due to congestion", address_with_type_.ToString().c_str());
       return;
     }