Re: [quicwg/base-drafts] QPACK [editorial] Update text to include both criteria of "blocking entry". (#2937)

[Bence Béky <[e-mail]>]

bencebeky commented on this pull request.



> @@ -229,8 +229,9 @@ table that it will emit a reference for.  As new entries are inserted, the
 encoder increases the draining index to maintain the section of the table that
 it will not reference.  If the encoder does not create new references to entries
 with an absolute index lower than the draining index, the number of
-unacknowledged references to those entries will eventually become zero, allowing
-them to be evicted.
+unacknowledged references to every such entry will eventually become zero,
+allowing it to be evicted provided that its insertion has been acknowledged by
+the decoder.

Ah, let me just revert the change to this paragraph.  No matter how I try to reword it, mentioning acknowledgement of the insertion always leaves it more confusing that the original text.  After all, if the number of unacknowledged references _becomes_ zero, that means it was not zero beforehand, therefore the encoder knows for sure that the insertion has been processed by the decoder.

-- 
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/2937#discussion_r314393645