From 8c8b5c3f8241c87e2beef491b28a8fd715e6f7a2 Mon Sep 17 00:00:00 2001 From: Rasmus Brandt Date: Tue, 11 Oct 2022 09:31:57 +0200 Subject: [PATCH] Logging clarification for inter_arrival_delta. Bug: b/250447844 Change-Id: I2a19f0a5c3fb58139ee1b07c47867bc0a1b49da6 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/278627 Reviewed-by: Per Kjellander Commit-Queue: Rasmus Brandt Cr-Commit-Position: refs/heads/main@{#38344} --- .../congestion_controller/goog_cc/inter_arrival_delta.cc | 6 +++--- modules/congestion_controller/goog_cc/inter_arrival_delta.h | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/modules/congestion_controller/goog_cc/inter_arrival_delta.cc b/modules/congestion_controller/goog_cc/inter_arrival_delta.cc index ecd5742272..2d50d08e6a 100644 --- a/modules/congestion_controller/goog_cc/inter_arrival_delta.cc +++ b/modules/congestion_controller/goog_cc/inter_arrival_delta.cc @@ -71,9 +71,9 @@ bool InterArrivalDelta::ComputeDeltas(Timestamp send_time, ++num_consecutive_reordered_packets_; if (num_consecutive_reordered_packets_ >= kReorderedResetThreshold) { RTC_LOG(LS_WARNING) - << "Packets between send burst arrived out of order, resetting." - << " arrival_time_delta" << arrival_time_delta->ms() - << " send time delta " << send_time_delta->ms(); + << "Packets between send burst arrived out of order, resetting:" + << " arrival_time_delta_ms=" << arrival_time_delta->ms() + << ", send_time_delta_ms=" << send_time_delta->ms(); Reset(); } return false; diff --git a/modules/congestion_controller/goog_cc/inter_arrival_delta.h b/modules/congestion_controller/goog_cc/inter_arrival_delta.h index 0617e34cdd..4046590eeb 100644 --- a/modules/congestion_controller/goog_cc/inter_arrival_delta.h +++ b/modules/congestion_controller/goog_cc/inter_arrival_delta.h @@ -42,7 +42,7 @@ class InterArrivalDelta { // `arrival_time` is the time at which the packet arrived. // `packet_size` is the size of the packet. // `timestamp_delta` (output) is the computed send time delta. - // `arrival_time_delta_ms` (output) is the computed arrival-time delta. + // `arrival_time_delta` (output) is the computed arrival-time delta. // `packet_size_delta` (output) is the computed size delta. bool ComputeDeltas(Timestamp send_time, Timestamp arrival_time,