Re: [xrblock] WGLC - draft-ietf-xrblock-rtcp-xr-decodability-02

Colin Perkins <csp@csperkins.org> Tue, 11 December 2012 21:16 UTC

Return-Path: <csp@csperkins.org>
X-Original-To: xrblock@ietfa.amsl.com
Delivered-To: xrblock@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0A9B311E80AE for <xrblock@ietfa.amsl.com>; Tue, 11 Dec 2012 13:16:34 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -106.599
X-Spam-Level:
X-Spam-Status: No, score=-106.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id w7R2IQ9SZMVm for <xrblock@ietfa.amsl.com>; Tue, 11 Dec 2012 13:16:33 -0800 (PST)
Received: from haggis.mythic-beasts.com (haggis.mythic-beasts.com [93.93.131.52]) by ietfa.amsl.com (Postfix) with ESMTP id 2C9DD11E80A6 for <xrblock@ietf.org>; Tue, 11 Dec 2012 13:16:33 -0800 (PST)
Received: from [217.108.124.185] (helo=[10.5.0.232]) by haggis.mythic-beasts.com with esmtpsa (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.72) (envelope-from <csp@csperkins.org>) id 1TiXBl-0003Ma-Ja; Tue, 11 Dec 2012 21:16:31 +0000
Mime-Version: 1.0 (Apple Message framework v1283)
Content-Type: text/plain; charset=windows-1252
From: Colin Perkins <csp@csperkins.org>
In-Reply-To: <9904FB1B0159DA42B0B887B7FA8119CA024828@AZ-FFEXMB04.global.avaya.com>
Date: Tue, 11 Dec 2012 22:16:14 +0100
Content-Transfer-Encoding: quoted-printable
Message-Id: <A9B61A34-690A-4F19-930E-CCBE3FC508CB@csperkins.org>
References: <9904FB1B0159DA42B0B887B7FA8119CA024828@AZ-FFEXMB04.global.avaya.com>
To: "Romascanu, Dan (Dan)" <dromasca@avaya.com>
X-Mailer: Apple Mail (2.1283)
X-BlackCat-Spam-Score: -12
X-Mythic-Debug: Threshold = On =
Cc: "xrblock@ietf.org" <xrblock@ietf.org>
Subject: Re: [xrblock] WGLC - draft-ietf-xrblock-rtcp-xr-decodability-02
X-BeenThere: xrblock@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Metric Blocks for use with RTCP's Extended Report Framework working group discussion list <xrblock.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/xrblock>, <mailto:xrblock-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/xrblock>
List-Post: <mailto:xrblock@ietf.org>
List-Help: <mailto:xrblock-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/xrblock>, <mailto:xrblock-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 11 Dec 2012 21:16:34 -0000

On 29 Nov 2012, at 13:48, Romascanu, Dan (Dan) wrote:
> This is a Working Group Last Call for http://www.ietf.org/id/draft-ietf-xrblock-rtcp-xr-decodability-02.txt.  
> 
> Please read and review this document, and send your comments, questions and concerns to the WG list before December 13, 2012. If you have no comments and you believe that the document is ready for submission to the IESG as a Standards Track document please send a short message as well to help us in determining the level of review and consensus. 


I have some minor, primarily editorial, comments below, but in general this looks okay.

- should expand “TS” -> “MPEG Transport Stream” in the title

- the first sentence of the Abstract could usefully be changed to “An MPEG Transport Stream (TS) is a...”

- change “this draft” to “this memo” throughout

- “occurances” -> “occurrences” in the Introduction

- The packet diagram at the end of page 4 says “BT=TBD” but the text at the top of page 5 says “...is identified by the constant <TDM>”. These should be consistent, and should match the value in Section 5.

- The description of SDP offer/answer usage in Section 4.2 is insufficient, and needs to be updated in a manner analogous to the other recent RTCP XR blocks.


-- 
Colin Perkins
http://csperkins.org/