Re: [quicwg/base-drafts] Fix Recovery Pseudocode (#2907)

[Jana Iyengar <[e-mail]>]

janaiyengar approved this pull request.

Thanks for the change -- looks MUCH better now.

>        time = loss_time[pn_space];
       space = pn_space
   return time, space
 
+PeerAwaitingAddressValidation():

```suggestion
IsPeerAwaitingAddressValidation():
```

>  SetLossDetectionTimer():
   loss_time, _ = GetEarliestLossTime()
   if (loss_time != 0):
     // Time threshold loss detection.
     loss_detection_timer.update(loss_time)
     return
 
-  // Don't arm timer if there are no ack-eliciting packets
-  // in flight and the handshake is complete.
-  if (endpoint is client with 1-RTT keys &&
-      no ack-eliciting packets in flight):
+  if (no ack-eliciting packets in flight &&
+      !PeerAwaitingAddressValidation()):

```suggestion
      !IsPeerAwaitingAddressValidation()):
```

-- 
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/pull/2907#pullrequestreview-283331379