From 1c94b969c784995640c69b9431fcc1fb71a9a745 Mon Sep 17 00:00:00 2001 From: Martin Thomson Date: Thu, 1 Oct 2020 10:02:42 +1000 Subject: [PATCH] Use brackets for empty arrays The curlies were fine, but we use brackets already for: ``` for pn_space in [ Initial, Handshake, ApplicationData ]: ``` So this is arguably more consistent. --- draft-ietf-quic-recovery.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/draft-ietf-quic-recovery.md b/draft-ietf-quic-recovery.md index 1b1179528b..ff2574dbf6 100644 --- a/draft-ietf-quic-recovery.md +++ b/draft-ietf-quic-recovery.md @@ -1544,7 +1544,7 @@ Pseudocode for DetectAndRemoveLostPackets follows: DetectAndRemoveLostPackets(pn_space): assert(largest_acked_packet[pn_space] != infinite) loss_time[pn_space] = 0 - lost_packets = {} + lost_packets = [] loss_delay = kTimeThreshold * max(latest_rtt, smoothed_rtt) // Minimum time of kGranularity before packets are deemed lost. @@ -1760,7 +1760,7 @@ OnPacketsLost(lost_packets): // packets indicates persistent congestion. // Only consider packets sent after getting an RTT sample. assert(first_rtt_sample != 0) - pc_lost = {} + pc_lost = [] for lost in lost_packets: if lost.time_sent > first_rtt_sample: pc_lost.insert(lost)