Re: [payload] I-D Action: draft-ietf-avt-rtp-isac-03.txt

Colin Perkins <csp@csperkins.org> Fri, 25 January 2013 12:06 UTC

Return-Path: <csp@csperkins.org>
X-Original-To: payload@ietfa.amsl.com
Delivered-To: payload@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 51B4B21F8A41 for <payload@ietfa.amsl.com>; Fri, 25 Jan 2013 04:06:02 -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 mcNWc9NUoJ8Z for <payload@ietfa.amsl.com>; Fri, 25 Jan 2013 04:06:01 -0800 (PST)
Received: from haggis.mythic-beasts.com (haggis.mythic-beasts.com [93.93.131.52]) by ietfa.amsl.com (Postfix) with ESMTP id 4B9C721F8862 for <payload@ietf.org>; Fri, 25 Jan 2013 04:06:01 -0800 (PST)
Received: from [130.209.247.112] (helo=mangole.dcs.gla.ac.uk) by haggis.mythic-beasts.com with esmtpsa (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.72) (envelope-from <csp@csperkins.org>) id 1Tyi2S-0005Zt-A2; Fri, 25 Jan 2013 12:05:44 +0000
Mime-Version: 1.0 (Apple Message framework v1283)
Content-Type: text/plain; charset="us-ascii"
From: Colin Perkins <csp@csperkins.org>
In-Reply-To: <5102697B.2030407@alvestrand.no>
Date: Fri, 25 Jan 2013 12:05:43 +0000
Content-Transfer-Encoding: quoted-printable
Message-Id: <6B0380A8-644E-492E-A921-6A18CEFE0B9D@csperkins.org>
References: <20130116214300.27292.59947.idtracker@ietfa.amsl.com> <B24AE5DD-7649-42B3-BD33-1169A169270E@csperkins.org> <5102697B.2030407@alvestrand.no>
To: Harald Alvestrand <harald@alvestrand.no>
X-Mailer: Apple Mail (2.1283)
X-BlackCat-Spam-Score: -13
X-Mythic-Debug: Threshold = On =
Cc: payload@ietf.org
Subject: Re: [payload] I-D Action: draft-ietf-avt-rtp-isac-03.txt
X-BeenThere: payload@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Audio/Video Transport Payloads working group discussion list <payload.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/payload>, <mailto:payload-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/payload>
List-Post: <mailto:payload@ietf.org>
List-Help: <mailto:payload-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/payload>, <mailto:payload-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 25 Jan 2013 12:06:02 -0000

On 25 Jan 2013, at 11:16, Harald Alvestrand wrote:
> On 01/17/2013 12:43 PM, Colin Perkins wrote:
>> On 16 Jan 2013, at 21:43, Internet-Drafts@ietf.org wrote:
>>> A New Internet-Draft is available from the on-line Internet-Drafts directories.
>>> This draft is a work item of the Audio/Video Transport Payloads Working Group of the IETF.
>>> 
>>> 	Title           : RTP Payload Format for the iSAC Codec
>>> 	Author(s)       : Tina le Grand
>>>                          Paul E. Jones
>>>                          Pascal Huart
>>>                          Turaj Zakizadeh Shabestary
>>>                          Harald Alvestrand
>>> 	Filename        : draft-ietf-avt-rtp-isac-03.txt
>>> 	Pages           : 15
>>> 	Date            : 2013-01-16
>>> 
>>> Abstract:
>>>   iSAC is a proprietary wideband speech and audio codec developed by
>>>   Global IP Solutions (now part of Google), suitable for use in Voice
>>>   over IP applications.  This document describes the payload format for
>>>   iSAC generated bit streams within a Real-Time Protocol (RTP) packet.
>>>   Also included here are the necessary details for the use of iSAC with
>>>   the Session Description Protocol (SDP).
>>> 
>>> The IETF datatracker status page for this draft is:
>>> https://datatracker.ietf.org/doc/draft-ietf-avt-rtp-isac
>> 
>> One quick comment: the examples in Sections 6.1 and 6.2 use RFC 2119 language ("SHOULD"), but I don't think an example is the right place to define normative requirements for implementations. Can the RFC 2119 terms be removed from these sections (and possibly be replaced by references to a section where the response is defined)?
>> n
> The normative language corrsponding to these examples is in section 5, where the parameters are defined. This language has been unchanged (including the use of uppercase MAY and SHOULD) since the July 2009 version of this draft, so the comment's definitely late.
> 
> I'll lowercase the terms in the next version; I don't see a need to do so before an IETF Last Call, but will take instructions from the chairs.


Just to be clear: no objections to normative language in Section 5, or in the start of Section 6 where the mapping is defined, but I believe it shouldn't be used in the examples in Sections 6.1 and 6.2.

And no, I don't see any need to do this before IETF last call. It's a nit after all.

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