[quicwg/base-drafts] Use "Insert Count" rather than "Largest Reference" (#2111)
Martin Thomson <notifications@github.com> Tue, 11 December 2018 07:06 UTC
Return-Path: <noreply@github.com>
X-Original-To: quic-issues@ietfa.amsl.com
Delivered-To: quic-issues@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 235671294D0 for <quic-issues@ietfa.amsl.com>; Mon, 10 Dec 2018 23:06:48 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -9.46
X-Spam-Level:
X-Spam-Status: No, score=-9.46 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-1.46, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=github.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id GRy1uLJP-okv for <quic-issues@ietfa.amsl.com>; Mon, 10 Dec 2018 23:06:46 -0800 (PST)
Received: from out-6.smtp.github.com (out-6.smtp.github.com [192.30.252.197]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 45D6A12DD85 for <quic-issues@ietf.org>; Mon, 10 Dec 2018 23:06:46 -0800 (PST)
Date: Mon, 10 Dec 2018 23:06:45 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1544512005; bh=Z1noY/eMHChnt3VJglzHpUbf6xrda6djuhLrp15YMYY=; h=Date:From:Reply-To:To:Cc:Subject:List-ID:List-Archive:List-Post: List-Unsubscribe:From; b=jryHgbZSbOVVb+kESq/N+O2EmpPwUam75AUvKmS0/30palmH9VV6WpRJxWhea9riO k4CajfObyx9rnqFX6SjGPQWd5wcQnLO89BueTROqCxGCmya6gMd/ZYBaP+qqb6TkHs q8AJs8vtCAfMHaUBBec5TuA6gmh6pIlE0fNA76ag=
From: Martin Thomson <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab31f40256b0c63dd6764c1ceffa5b0deff37aa72792cf000000011827240592a169ce17392ee3@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2111@github.com>
Subject: [quicwg/base-drafts] Use "Insert Count" rather than "Largest Reference" (#2111)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c0f620536864_51483ffd1f2d45b86122a7"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: martinthomson
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address: quic-issues@ietf.org
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/sTVKt1DlUpBjMQFAMtXUbCFj2Os>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Notification list for GitHub issues related to the QUIC WG <quic-issues.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/quic-issues/>
List-Post: <mailto:quic-issues@ietf.org>
List-Help: <mailto:quic-issues-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 11 Dec 2018 07:06:48 -0000
The construction of largest reference currently implies the use of 1-based indexing, which is unnatural. Using the insert count is much more natural, even if it means the encoding is a little strange and a tiny bit inefficient (it goes 0, 2, 3, ..., Max-1, 1, 2, ...). I've gone through and put the references on the boundary between records, which makes explanations much easier for post-base indexing in my view. I've renamed Base Index (which was never really an index) to Base, which matches that change. Similarly, renaming Largest Known Received to Known Received Count makes it match better. There are also examples added to Insert Count and Base sections. These definitions are pretty damned opaque without them. Closes #2110. You can view, comment on, or merge this pull request online at: https://github.com/quicwg/base-drafts/pull/2111 -- Commit Summary -- * Use "Insert Count" rather than "Largest Reference" -- File Changes -- M draft-ietf-quic-qpack.md (257) -- Patch Links -- https://github.com/quicwg/base-drafts/pull/2111.patch https://github.com/quicwg/base-drafts/pull/2111.diff -- 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/2111
- [quicwg/base-drafts] Use "Insert Count" rather th… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… afrind
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Mike Bishop
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… afrind
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Bence Béky
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Mike Bishop
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Mike Bishop
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… afrind
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… afrind
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… afrind
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Martin Thomson
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… Mike Bishop
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… afrind
- Re: [quicwg/base-drafts] Use "Insert Count" rathe… afrind