From baf1512a4c6b461fd8c4b21fa95e4046ab9a05ec Mon Sep 17 00:00:00 2001 From: Jakob Ivarsson Date: Mon, 1 Nov 2021 10:14:29 +0100 Subject: [PATCH] Change back kDefaultMaxReorderingThreshold to 50 packets. This was changed by mistake (?) to 5 in a refactoring cl: https://webrtc-review.googlesource.com/c/src/+/222324 This caused the packets lost metric to not count loss gaps that are larger than 5 packets. Bug: webrtc:13336 Change-Id: Ied4732312aeed81862a74fbc889e33fcedde3def Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/236840 Commit-Queue: Jakob Ivarsson Reviewed-by: Danil Chapovalov Cr-Commit-Position: refs/heads/main@{#35290} --- modules/rtp_rtcp/source/rtp_rtcp_config.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/modules/rtp_rtcp/source/rtp_rtcp_config.h b/modules/rtp_rtcp/source/rtp_rtcp_config.h index 66caadd578..3e6aa3baae 100644 --- a/modules/rtp_rtcp/source/rtp_rtcp_config.h +++ b/modules/rtp_rtcp/source/rtp_rtcp_config.h @@ -15,7 +15,7 @@ // Configuration file for RTP utilities (RTPSender, RTPReceiver ...) namespace webrtc { -constexpr int kDefaultMaxReorderingThreshold = 5; // In sequence numbers. +constexpr int kDefaultMaxReorderingThreshold = 50; // In sequence numbers. constexpr int kRtcpMaxNackFields = 253; constexpr TimeDelta RTCP_SEND_BEFORE_KEY_FRAME = TimeDelta::Millis(100);