Re: [quicwg/base-drafts] QPACK [editorial] Misc minor editorial changes. (#2942)

Bence Béky <> Wed, 31 July 2019 17:00 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id DB8A51204C3 for <>; Wed, 31 Jul 2019 10:00:36 -0700 (PDT)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: -1.381
X-Spam-Status: No, score=-1.381 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FROM_EXCESS_BASE64=0.001, GB_SUMOF=5, HTML_IMAGE_ONLY_24=1.618, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=no autolearn_force=no
Authentication-Results: (amavisd-new); dkim=pass (1024-bit key)
Received: from ([]) by localhost ( []) (amavisd-new, port 10024) with ESMTP id MxlLN42Okdk4 for <>; Wed, 31 Jul 2019 10:00:35 -0700 (PDT)
Received: from ( []) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by (Postfix) with ESMTPS id 755261201EE for <>; Wed, 31 Jul 2019 10:00:35 -0700 (PDT)
Date: Wed, 31 Jul 2019 10:00:34 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=pf2014; t=1564592434; bh=12IssdTG32sjR+n6LS78qf+oEwvb/sGsQkuHNQsIF+Y=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=GYzShkUkxt7aAVsXV7TbixPAq+1avAS7ncfvU/zKkziRgdx975x/kEpyB7aJxy4MA 3t5JK5821gtv58km7g6Yyk0AcYeYhzHWGXAXERkrZqk4oZQx0ZQkwCIOB/VG1hs1ZI 9YDXumtWReVe8oOh9SeJhnZjk2imx0aojim06dfc=
From: =?UTF-8?B?QmVuY2UgQsOpa3k=?= <>
Reply-To: quicwg/base-drafts <>
To: quicwg/base-drafts <>
Cc: Subscribed <>
Message-ID: <quicwg/base-drafts/pull/2942/review/>
In-Reply-To: <quicwg/base-drafts/pull/>
References: <quicwg/base-drafts/pull/>
Subject: Re: [quicwg/base-drafts] QPACK [editorial] Misc minor editorial changes. (#2942)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5d41c93292869_21583fd8106cd9641668ad"; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: bencebeky
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
Archived-At: <>
X-Mailman-Version: 2.1.29
List-Id: Notification list for GitHub issues related to the QUIC WG <>
List-Unsubscribe: <>, <>
List-Archive: <>
List-Post: <>
List-Help: <>
List-Subscribe: <>, <>
X-List-Received-Date: Wed, 31 Jul 2019 17:00:37 -0000

bencebeky commented on this pull request.

> @@ -430,12 +431,9 @@ Dynamic table entries can have empty values.
 The size of the dynamic table is the sum of the size of its entries.
-The size of an entry is the sum of its name's length in bytes (as defined in
-{{string-literals}}), its value's length in bytes, and 32.

The reference to {{string-literals}} here is misleading, because length in that section can mean length after Huffman encoding.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub: