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: hurtta@hurtta09lk.keh.iki.fi [mailto:hurtta@hurtta09lk.keh.iki.fi] 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#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
> Received on Friday, 11 July 2014 10:35:27 UTC