IETF Logo Mail Archive

Re: [quicwg/base-drafts] QPACK [editorial] update RIC and Base on figures (#2935)

afrind <notifications@github.com> Tue, 13 August 2019 21:29 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 CF2E11208C2 for <quic-issues@ietfa.amsl.com>; Tue, 13 Aug 2019 14:29:43 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.454
X-Spam-Level:
X-Spam-Status: No, score=-6.454 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, HTML_IMAGE_ONLY_20=1.546, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_HELO_NONE=0.001, 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 TovfB32nbMFm for <quic-issues@ietfa.amsl.com>; Tue, 13 Aug 2019 14:29:42 -0700 (PDT)
Received: from out-19.smtp.github.com (out-19.smtp.github.com [192.30.252.202]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 7FBC01208E6 for <quic-issues@ietf.org>; Tue, 13 Aug 2019 14:29:42 -0700 (PDT)
Date: Tue, 13 Aug 2019 14:29:41 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1565731781; bh=DYWzClIy8DBRMxAWRVgFxdRKSPt07Ct/74PQNVU2ZHY=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=papFvfxSxnUp7uaVBFRlGbeYBGjpdk87FgodzWVaUKcL6pEC98kClRJ3NgBkPCqSk FbDLDT64cyAUDJ7CcxaxTwWsTmSGHUeJGyHn+oVgIKwQ6UXLvJ+7qWQ4kg1Tmwe+2d ouVsRq1StOBTA7CRAhMN2GatqcW+Fxdz5gKkZz0g=
From: afrind <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJKY6DYYNZ45WIU36LP53MBPELEVBNHHBYU2QL4@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2935/c521016601@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2935@github.com>
References: <quicwg/base-drafts/pull/2935@github.com>
Subject: Re: [quicwg/base-drafts] QPACK [editorial] update RIC and Base on figures (#2935)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5d532bc59725f_25cc3fe5602cd9609995f0"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: afrind
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/Sslo4zGcSHVv0MzwLp1HLP1Tpmc>
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, 13 Aug 2019 21:29:44 -0000

Agree we should remove the arrow from count.  I guess there's two minds on where array indices point.  I think I inherited the current style.  HPACK only has the 'insertion point' and 'dropping point' arrows, so sidesteps the issue altogether. 

I'm inlined to say it's more clear as is, with Base pointing on the line between what was in the table when encoding started and the Post Base entries.

-- 
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/2935#issuecomment-521016601

v2.23.0 | Report a Bug | By Email