Subject: RE: draft-ietf-httpbis-header-compression-latest, 4.1 Header Field Representation Processing

From: RUELLAN Herve <[email-masked]>
Date: Fri, 11 Jul 2014 10:34:33 +0000

I agree that read in isolation, section 4.1 seems to contradict itself.

I'm going to improve the wording to make this cleared.

Thanks for pointing at this.

Hervé.

> -----Original Message-----
> From: [email-masked] [mailto:[email-masked]] On
> Behalf Of Kari Hurtta
> Sent: vendredi 11 juillet 2014 06:23
> To: Roberto Peon; RUELLAN Herve
> Cc: Kari Hurtta; HTTPBIS working group mailing list
> Subject: draft-ietf-httpbis-header-compression-latest, 4.1 Header Field
> Representation Processing
> 
> HPACK - Header Compression for HTTP/2
> draft-ietf-httpbis-header-compression-latest
> July 7, 2014
> http://http2.github.io/http2-spec/compression.html
> 
> 
> 4.1 Header Field Representation Processing
> http://http2.github.io/http2-
> spec/compression.html#header.representation.processing
> 
> 
> This quite strange (and difficult to understand) text
> 
> » • If referencing an element of the static table:
> »
> »     • The header field corresponding to the referenced entry is emitted.
> »     • The referenced static entry is inserted at the beginning of
> »       the header table.
> »     • A reference to this new header table entry is added to the
> »       reference set, unless this new entry didn't fit in the header table.
> 
> First
> 
>      entry is inserted at the header table.
> 
> but then
> 
>      "unless this new entry didn't fit in the header table"
> 
> when first it said that it was inserted.
> 
> Should this be
> 
> |    unless this new entry was immediately evicted from header table
> |    (resulting empty table).
> 
> 
> 
> and on another place
> 
> » •  The header field is inserted at the beginning of the header table.
> » •  A reference to the new entry is added to the reference set, unless
> »    this new entry didn't fit in the header table.
> 
> Also here first
> 
>      header field is inserted at the header table.
> 
> but then
> 
>      "unless his new entry didn't fit in the header table."
> 
> should this be
> 
> |    unless this new entry was immediately evicted from header table
> |    (resulting empty table).
> 
> 
> 
> 5.3   Entry Eviction when Adding New Entries
> http://http2.github.io/http2-spec/compression.html#entry.addition
> 
> » Whenever a new entry is to be added to the header table entries are
> » evicted from the end of the header table until the size of the header
> » table is less than or equal to (maximum size - new entry size), or
> » until the table is empty.
> 
> 
> / Kari Hurtta