[quicwg/base-drafts] Pseudocode fix: Data should be sent on loss-threshold timer (#2604)

[Jana Iyengar <[e-mail]>]

(@larseggert : Thanks for catching this)

Currently, in OnLossDetectionTimeout(), we detect losses but do not sent anything on time threshold loss detection.  This means that SetLossDetectionTimer() that is called later in the function does not set a timer because inflight can be 0 (if all inflight is marked lost).

This is an oversight, since clearly we send rtxs when we mark packets as lost, and in the case of threshold detection, we send as much as the cwnd allows.

My proposal is to add the last two lines below
if (loss_time != 0):
    // Time threshold loss Detection
    DetectLostPackets(pn_space)
    SendOnePacketWithRetransmittedData()
    SendDataAsCwndAllows()

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/issues/2604