Re: [quicwg/base-drafts] Don't call two different things "blocking" (#3172)

[Dmitri Tikhonov <[e-mail]>]

dtikhonov requested changes on this pull request.

This review _suggests_ changes, not "requires" them.

>  
 An encoder MUST NOT insert an entry into the dynamic table (or duplicate an
-existing entry) if doing so would evict a blocking entry.  In order to avoid
-this, an encoder that uses the dynamic table has to keep track of blocking
-entries.
+existing entry) unless all entries that would be evicted are evictable.  In

That a bit of a roundabout way of stating that encoder might not be able to free up enough room due to non-evictable entries.

Perhaps instead of ".. unless..."

> ...if the dynamic table does not have enough room and one or more entries that would need to be evicted to make room are not evictable.

>  
-A dynamic table entry is considered blocking and cannot be evicted until its
-insertion has been acknowledged and there are no outstanding unacknowledged
-references to the entry.  In particular, a dynamic table entry that has never
-been referenced can still be blocking.
+A dynamic table entry cannot be evicted immediately after insertion, even if it

The heading says this section is about limits on insertions, yet it begins with limits on evictions.  It would be better to start with something like

> Insertion of the dynamic table entry may not be possible if the table contains entries that cannot be evicted.  [Explanation follows.]

-- 
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/3172#pullrequestreview-309404880