Re: [quicwg/base-drafts] Update PTO pseudocode to better match text (#3461)

[Martin Thomson <[e-mail]>]

martinthomson approved this pull request.



> +    // There is nothing to detect lost and the server
+    // is not blocked by the anti-amplification limit.

This comment is a little strange.

The point here is to catch the case where there is nothing to send, with the exception that the client might need to keep the server primed in case it is blocked on its anti-amplification limit.

As we've already dispensed with the anti-amplification limit at the server, I think that this needs to concentrate on the client instead.

```suggestion
    // There is nothing to detect lost, so set no timer.
    // However, the client still sets the timer if the server
    // might be blocked by the anti-amplification limit.
```

-- 
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/3461#pullrequestreview-374835865