Re: [quicwg/base-drafts] add whitespace after semicolon in static header table (#1984)

afrind <notifications@github.com> Tue, 27 November 2018 23:22 UTC

Date: Tue, 27 Nov 2018 15:22:07 -0800
From: afrind <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abfd29d760ec9ed548ab7a2f42d471cb72c871752592cf000000011815939f92a169ce1698914e@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/1984/c442256652@github.com>
In-Reply-To: <quicwg/base-drafts/pull/1984@github.com>
References: <quicwg/base-drafts/pull/1984@github.com>
Subject: Re: [quicwg/base-drafts] add whitespace after semicolon in static header table (#1984)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5bfdd19f3bd0e_7dd93f9fd12d45b42207b4"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/10GvAPwxmjOL86sVcdBjNoO_m74>

I think it's pretty unlikely anyone is going to figure out that they can save bytes on the wire by adding a space to their header value.  We decided to use the real world to guide the contents of the table.  In other cases where we felt the table was encouraging bad practice, we dropped the header from the table.  But if this one is "too important" to drop, the it's probably "too important" to change from common to canonical usage for the sake of consistency or encouraging better practice.

-- 
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/1984#issuecomment-442256652

[quicwg/base-drafts] add whitespace after semicol… Kazuho Oku
Re: [quicwg/base-drafts] add whitespace after sem… Dmitri Tikhonov
Re: [quicwg/base-drafts] add whitespace after sem… Mike Bishop
Re: [quicwg/base-drafts] add whitespace after sem… Mike Bishop
Re: [quicwg/base-drafts] add whitespace after sem… Kazuho Oku
Re: [quicwg/base-drafts] add whitespace after sem… MikkelFJ
Re: [quicwg/base-drafts] add whitespace after sem… Kazuho Oku
Re: [quicwg/base-drafts] add whitespace after sem… afrind
Re: [quicwg/base-drafts] add whitespace after sem… Martin Thomson
Re: [quicwg/base-drafts] add whitespace after sem… Kazuho Oku
Re: [quicwg/base-drafts] add whitespace after sem… MikkelFJ
Re: [quicwg/base-drafts] add whitespace after sem… afrind
Re: [quicwg/base-drafts] add whitespace after sem… afrind