From 7daa6462428e3bda9313451976223c156bc05866 Mon Sep 17 00:00:00 2001 From: ianswett Date: Tue, 10 Sep 2019 14:27:22 -0400 Subject: [PATCH] Use contains() Per @martinthomson suggestion, use contains() instead of the implicit check. Fixes #2569 An alternative to #2987 --- draft-ietf-quic-recovery.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/draft-ietf-quic-recovery.md b/draft-ietf-quic-recovery.md index 39658cb175..9a497ea40f 100644 --- a/draft-ietf-quic-recovery.md +++ b/draft-ietf-quic-recovery.md @@ -1097,7 +1097,7 @@ OnAckReceived(ack, pn_space): // If the largest acknowledged is newly acked and // at least one ack-eliciting was newly acked, update the RTT. - if (sent_packets[pn_space][ack.largest_acked] && + if (sent_packets[pn_space].contains(ack.largest_acked) && IncludesAckEliciting(newly_acked_packets)): latest_rtt = now - sent_packets[pn_space][ack.largest_acked].time_sent