[PATCH] HTTP/2: HPACK Huffman encoding
Vlad Krasnov
vlad at cloudflare.com
Thu Dec 17 11:40:14 UTC 2015
I will remove the tabs, not sure how they got there.
Regarding the minimal compression field: encoding 4 bytes yields real benefits. Values like “gzip” or “link” get one byte shoved off, which makes it 25% smaller and it adds up.
Less than 4 bytes is probably useless, but then again those are not common, and shouldn’t be a performance issue.
> On 17 Dec 2015, at 11:30, Valentin V. Bartenev <vbart at nginx.com> wrote:
>
> On Thursday 17 December 2015 01:36:07 Vlad Krasnov wrote:
>> # HG changeset patch
>> # User Vlad Krasnov <vlad at cloudflare.com>
>> # Date 1450274269 28800
>> # Wed Dec 16 05:57:49 2015 -0800
>> # Node ID d2e16044797ef7f7e0583e7c6dfdae5402c70d5c
>> # Parent def9c9c9ae05cfa7467b0ec96e76afa180c23dfb
>> HTTP/2: HPACK Huffman encoding
>>
>> Implement HPACK Huffman encoding for HTTP/2.
>> This reduces the size of headers by over 30% on average.
>>
> [..]
>
> Thank you for the patch. I'll look at it as time permits.
>
> From the brief glance I've spotted a number of style issues.
> The most obvious is mixed tabs and spaces in indentation.
>
> Also I think we shouldn't even try to compress values less
> than 4 bytes (or even more... like 8-16, IMO the probability
> of saving 1-4 bytes doesn't worth it).
>
> wbr, Valentin V. Bartenev
>
> _______________________________________________
> nginx-devel mailing list
> nginx-devel at nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-devel
More information about the nginx-devel
mailing list