Skip to content

Commit

Permalink
pw_bluetooth_proxy: Remove unnecessary writer for disconnect event
Browse files Browse the repository at this point in the history
Can just be normal view since we don't need to rewrite. This also
improves verification of incoming data.

Change-Id: Ib500dc7f2687ec31cd64f34d87d5b3d0a660d60f
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/249716
Reviewed-by: Ben Lawson <[email protected]>
Commit-Queue: Austin Foxley <[email protected]>
Lint: Lint 🤖 <[email protected]>
Docs-Not-Needed: Ben Lawson <[email protected]>
  • Loading branch information
afoxley authored and CQ Bot Account committed Nov 22, 2024
1 parent 8a31009 commit 22c0113
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pw_bluetooth_proxy/acl_data_channel.cc
Original file line number Diff line number Diff line change
Expand Up @@ -189,8 +189,8 @@ void AclDataChannel::HandleNumberOfCompletedPacketsEvent(

void AclDataChannel::HandleDisconnectionCompleteEvent(
H4PacketWithHci&& h4_packet) {
Result<emboss::DisconnectionCompleteEventWriter> dc_event =
MakeEmbossView<emboss::DisconnectionCompleteEventWriter>(
Result<emboss::DisconnectionCompleteEventView> dc_event =
MakeEmbossView<emboss::DisconnectionCompleteEventView>(
h4_packet.GetHciSpan());
if (!dc_event.ok()) {
PW_LOG_ERROR(
Expand Down

0 comments on commit 22c0113

Please sign in to comment.