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 6769A12006F
 for <quic-issues@ietfa.amsl.com>; Fri, 27 Dec 2019 07:58:25 -0800 (PST)
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 obNIVLF1kFrV for <quic-issues@ietfa.amsl.com>;
 Fri, 27 Dec 2019 07:58:23 -0800 (PST)
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 BA94C120019
 for <quic-issues@ietf.org>; Fri, 27 Dec 2019 07:58:23 -0800 (PST)
Date: Fri, 27 Dec 2019 07:58:23 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
 s=pf2014; t=1577462303;
 bh=Xd3VmNi7YKXnvqSf6XnCsff/YqbpxaCiwKe2amTRFSI=;
 h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID:
 List-Archive:List-Post:List-Unsubscribe:From;
 b=bEEK4IMddXacOXEkyLkj+PMQ85dWpXu+4vOL5hrTdBUFmT+58XDwvV4vQFS4os+LI
 NYxz8QRQ2G+PDYXd3mkiQoij7JvE8vV2ywN4Q1XlDCEMrmu125DIaMCjG0En9hTFP9
 PewefUEyogVIiy6mc26ne0WsvAvMgHIQwYNi5Cw4=
From: Dmitri Tikhonov <notifications@github.com>
Reply-To: quicwg/base-drafts
 <reply+AFTOJK4GQBHL6J3AON2223F4CNOJ7EVBNHHCAXBUGE@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/3310/569296381@github.com>
In-Reply-To: <quicwg/base-drafts/issues/3310@github.com>
References: <quicwg/base-drafts/issues/3310@github.com>
Subject: Re: [quicwg/base-drafts] Unnecessary packet number expectations in
 ACK frame description (#3310)
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_5e062a1f1421e_5573fdfe76cd96c3073e1";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: dtikhonov
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/zkaO9Ak6lcQijRKeBZ7audMbuY0>
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: Fri, 27 Dec 2019 15:58:26 -0000


----==_mimepart_5e062a1f1421e_5573fdfe76cd96c3073e1
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

The second sentence can stand some improvement as well:
1. There is no need for the word "only."
1. It is not necessary to say that the ACK frame was received.
1. Packets, not packet numbers, are acknowledged.

How about something like this:
> An ACK frame acknowledges packets in the same packet number space as the packet in which it is contained.

-- 
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/issues/3310#issuecomment-569296381
----==_mimepart_5e062a1f1421e_5573fdfe76cd96c3073e1
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

<p>The second sentence can stand some improvement as well:</p>
<ol>
<li>There is no need for the word "only."</li>
<li>It is not necessary to say that the ACK frame was received.</li>
<li>Packets, not packet numbers, are acknowledged.</li>
</ol>
<p>How about something like this:</p>
<blockquote>
<p>An ACK frame acknowledges packets in the same packet number space as the packet in which it is contained.</p>
</blockquote>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/quicwg/base-drafts/issues/3310?email_source=notifications&amp;email_token=AFTOJKYZETQPDWICVRR5UPLQ2YQZ7A5CNFSM4J76ADN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHXML7I#issuecomment-569296381">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AFTOJKYKJWTOROG67QPEDQDQ2YQZ7ANCNFSM4J76ADNQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AFTOJK3O4EFNNQHYLQGBTYTQ2YQZ7A5CNFSM4J76ADN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHXML7I.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/quicwg/base-drafts/issues/3310?email_source=notifications\u0026email_token=AFTOJKYZETQPDWICVRR5UPLQ2YQZ7A5CNFSM4J76ADN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHXML7I#issuecomment-569296381",
"url": "https://github.com/quicwg/base-drafts/issues/3310?email_source=notifications\u0026email_token=AFTOJKYZETQPDWICVRR5UPLQ2YQZ7A5CNFSM4J76ADN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHXML7I#issuecomment-569296381",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>
----==_mimepart_5e062a1f1421e_5573fdfe76cd96c3073e1--

