[quicwg/base-drafts] Improve language on blocking entries. (#3131)

Bence Béky <notifications@github.com> Tue, 22 October 2019 23:36 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 65B18120122 for <quic-issues@ietfa.amsl.com>; Tue, 22 Oct 2019 16:36:54 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -7.999
X-Spam-Level:
X-Spam-Status: No, score=-7.999 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_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=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 9njvKA8MBb1B for <quic-issues@ietfa.amsl.com>; Tue, 22 Oct 2019 16:36:52 -0700 (PDT)
Received: from out-22.smtp.github.com (out-22.smtp.github.com [192.30.252.205]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 20E4312006A for <quic-issues@ietf.org>; Tue, 22 Oct 2019 16:36:51 -0700 (PDT)
Received: from github-lowworker-f045d1f.ac4-iad.github.net (github-lowworker-f045d1f.ac4-iad.github.net [10.52.19.54]) by smtp.github.com (Postfix) with ESMTP id 264DBA0D79 for <quic-issues@ietf.org>; Tue, 22 Oct 2019 16:36:51 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1571787411; bh=uZLhgoRAUzMMVwUnTMP6DK/G+wot5YHQw+wkMMbrvLY=; h=Date:From:Reply-To:To:Cc:Subject:List-ID:List-Archive:List-Post: List-Unsubscribe:From; b=VRy7W0P2rUWP/sBWhTetzMRKRVD5jxs+pSsJK9SsBtM3l0BpT0KAsmykgrnAKOCZk 7WcF9z9XHcph08v7yHhVVi8IpIgDbCfWlf8Zv5i3QdMZD8zm66IL/Ky2Tv1t07x/Kh r9i8Nitw+sbKJbOV2p1yfqhRs2OZj+D+FKnL2SWY=
Date: Tue, 22 Oct 2019 16:36:51 -0700
From: Bence Béky <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK7ZRVWIB4GG2PQ3VBN3XTJSHEVBNHHB45IJ64@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/3131@github.com>
Subject: [quicwg/base-drafts] Improve language on blocking entries. (#3131)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5daf929317087_366e3fa2b86cd960160016"; 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
X-GitHub-Recipient-Address: quic-issues@ietf.org
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/aGj6Y6OPCAk_3pFBaY9uO-0sQU8>
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, 22 Oct 2019 23:36:54 -0000

Remove &quot;insertion has been acknowledged&quot; phrase to avoid confusion
between Header Acknowledgement instructions implicitly acknowledging
encoder stream instructions and explicitly acknowledging header blocks.
Use Known Received Count instead.

Clarify that references on the encoder stream do not block an entry from
eviction.  Trivial as it is, I overlooked this at first, and added quite
some unnecessary complexity that was eventually removed at
https://quiche.googlesource.com/quiche/+/b09d44167c543b80b2c27408dd1f9df3b07c9741.
My motivation here is to help future implementers to avoid this pitfall.

Merge redundant Reference Tracking section into Blocked Dynamic Table
Insertions.

Change style of &quot;Note&quot; at end of section for consistency.  There were
already four sentences starting with &quot;Note that&quot; throughout the draft
before this PR (now there are six), but no other &quot;Note:&quot; markdown.
You can view, comment on, or merge this pull request online at:

  https://github.com/quicwg/base-drafts/pull/3131

-- Commit Summary --

  * Improve language on blocking entries.

-- File Changes --

    M draft-ietf-quic-qpack.md (28)

-- Patch Links --

https://github.com/quicwg/base-drafts/pull/3131.patch
https://github.com/quicwg/base-drafts/pull/3131.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/3131