[PATCH] HTTP/2: HPACK Huffman encoding
Valentin V. Bartenev
vbart at nginx.com
Thu Dec 17 11:30:41 UTC 2015
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
More information about the nginx-devel
mailing list