Subject: [quicwg/base-drafts] SetLossDetectionAlarm issue with TLP (#537)

From: Lars Eggert <[email-masked]>
Date: Thu, 18 May 2017 02:59:42 -0700

The pseudo code in Section 3.2.7.4 has this conditional `return` in the beginning:
```
SetLossDetectionAlarm():
    if (retransmittable packets are not outstanding):
      loss_detection_alarm.cancel()
      return
```
Afterwards, in the TLP case, is says:
```
      // Tail Loss Probe
      if (retransmittable_packets_outstanding = 1):
        alarm_duration = 1.5 * smoothed_rtt + kDelayedAckTimeout
      else:
        alarm_duration = kMinTLPTimeout
      alarm_duration = max(alarm_duration, 2 * smoothed_rtt)
```
Given that the code would already have returned (per above) if `retransmittable_packets_outstanding` is false, how can the `else` clause here ever be executed?

-- 
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/537