IETF Logo Mail Archive

[payload] Gen-ART review of draft-ietf-payload-rtp-opus-08

Christer Holmberg <christer.holmberg@ericsson.com> Sat, 11 April 2015 13:23 UTC

Return-Path: <christer.holmberg@ericsson.com>
X-Original-To: expand-draft-ietf-payload-rtp-opus.all@virtual.ietf.org
Delivered-To: payload@ietfa.amsl.com
Received: by ietfa.amsl.com (Postfix, from userid 65534) id 334F51AD289; Sat, 11 Apr 2015 06:23:15 -0700 (PDT)
X-Original-To: xfilter-draft-ietf-payload-rtp-opus.all@ietfa.amsl.com
Delivered-To: xfilter-draft-ietf-payload-rtp-opus.all@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 12A901AD272 for <xfilter-draft-ietf-payload-rtp-opus.all@ietfa.amsl.com>; Sat, 11 Apr 2015 06:23:15 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.634
X-Spam-Level:
X-Spam-Status: No, score=-0.634 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, J_CHICKENPOX_14=0.6, SPF_SOFTFAIL=0.665] autolearn=no
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 TKdN1g-3eXvp for <xfilter-draft-ietf-payload-rtp-opus.all@ietfa.amsl.com>; Sat, 11 Apr 2015 06:23:13 -0700 (PDT)
Received: from zinfandel.tools.ietf.org (zinfandel.tools.ietf.org [IPv6:2001:1890:123a::1:2a]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 047B11AD17F for <draft-ietf-payload-rtp-opus.all@ietf.org>; Sat, 11 Apr 2015 06:23:13 -0700 (PDT)
Received: from sesbmg23.ericsson.net ([193.180.251.37]:46731) by zinfandel.tools.ietf.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.82_1-5b7a7c0-XX) (envelope-from <christer.holmberg@ericsson.com>) id 1YgvNQ-0002l5-0S for draft-ietf-payload-rtp-opus.all@tools.ietf.org; Sat, 11 Apr 2015 06:23:12 -0700
X-AuditID: c1b4fb25-f79126d000004b89-ee-5529203c828e
Received: from ESESSHC009.ericsson.se (Unknown_Domain [153.88.253.124]) by sesbmg23.ericsson.net (Symantec Mail Security) with SMTP id DD.1F.19337.C3029255; Sat, 11 Apr 2015 15:23:08 +0200 (CEST)
Received: from ESESSMB209.ericsson.se ([169.254.9.236]) by ESESSHC009.ericsson.se ([153.88.183.45]) with mapi id 14.03.0210.002; Sat, 11 Apr 2015 15:23:07 +0200
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: "gen-art@ietf.org" <gen-art@ietf.org>
Thread-Topic: Gen-ART review of draft-ietf-payload-rtp-opus-08
Thread-Index: AdB0YZrD1UZTCkzGSwKoMQ0d3L+1AA==
Date: Sat, 11 Apr 2015 13:23:05 +0000
Message-ID: <7594FB04B1934943A5C02806D1A2204B1D7939BB@ESESSMB209.ericsson.se>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.150]
Content-Type: multipart/alternative; boundary="_000_7594FB04B1934943A5C02806D1A2204B1D7939BBESESSMB209erics_"
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrFLMWRmVeSWpSXmKPExsUyM+Jvja6NgmaowddH4hZbtu9lsrj66jOL A5PHkiU/mTy+XP7MFsAUxWWTkpqTWZZapG+XwJWxYdF1poKb5hVTNi9gamB8btDFyMEhIWAi 0btatouRE8gUk7hwbz1bFyMXh5DAUUaJczuvsIAkhASWMEos3WQHUs8mYCHR/U8bJCwioCkx d8VbJpAws0C6xLlOKZCwsIClxKRtLSwQJXYS1+bcZYew9SSWNLYwgdgsAqoSqw79AKvhFfCV eLqnHSzOCHTC91NrwGxmAXGJW0/mM0GcJiCxZM95ZghbVOLl43+sELaSxIrtlxgh6vMl3p9a ywYxU1Di5MwnLBMYhWchGTULSdksJGUQcR2JBbs/sUHY2hLLFr5mhrHPHHjMhCy+gJF9FaNo cWpxUm66kbFealFmcnFxfp5eXmrJJkZg5Bzc8lt1B+PlN46HGAU4GJV4eBUaNEKFWBPLiitz DzFKc7AoifPaGR8KERJITyxJzU5NLUgtii8qzUktPsTIxMEp1cCYnB8hMSu8++X96LK1JclC i9OL1j+a1nd+98Ucnlse2uZyoYWh2kIzOYL/rzrZKxC34G9/MXdq4bQXl1lXMr0Wkn92V37N KrfPexYtuMTGcahZ9961YsZ7v6J3yuo1yZnkLbvreaFVrqrALulnWunlc0WztN/uSZrZoZKS ++BXDt+uk/Z7dgspsRRnJBpqMRcVJwIA7w26t30CAAA=
X-SA-Exim-Connect-IP: 193.180.251.37
X-SA-Exim-Rcpt-To: draft-ietf-payload-rtp-opus.all@tools.ietf.org
X-SA-Exim-Mail-From: christer.holmberg@ericsson.com
X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000)
X-SA-Exim-Scanned: Yes (on zinfandel.tools.ietf.org)
Resent-To: draft-ietf-payload-rtp-opus.all@ietf.org
Resent-Message-Id: <20150411132313.047B11AD17F@ietfa.amsl.com>
Resent-Date: Sat, 11 Apr 2015 06:23:13 -0700
Resent-From: christer.holmberg@ericsson.com
Archived-At: <http://mailarchive.ietf.org/arch/msg/draft-ietf-payload-rtp-opus.all@tools/YrDyc7IARvhdzyDMlO2WOA_upHs>
Archived-At: <http://mailarchive.ietf.org/arch/msg/payload/RnMEEzIGFZCpCi3ukPqbleoDhF8>
X-Mailman-Approved-At: Sat, 11 Apr 2015 09:03:20 -0700
Cc: "draft-ietf-payload-rtp-opus.all@tools.ietf.org" <draft-ietf-payload-rtp-opus.all@tools.ietf.org>
Subject: [payload] Gen-ART review of draft-ietf-payload-rtp-opus-08
X-BeenThere: payload@ietf.org
X-Mailman-Version: 2.1.15
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: Sat, 11 Apr 2015 13:23:15 -0000

I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>

Document:                         draft-ietf-payload-rtp-opus-08

Reviewer:                           Christer Holmberg

Review Date:                     11 April 2015

IETF LC End Date:             17 February 2015

IETF Telechat Date:         9 April 2015

Summary:        Most of the comments I gave on a previous version of the document have been addressed. However, it seems like one of the comments have not been addressed.

Major Issues: None

Minor Issues: I previously gave the following comment:


"Regarding SDP, I think it would be good to have the ABNF syntax for the a=fmtp parameter (currently you only have descriptive text of the different parameters). It makes the life for the parser implementers much easier :)"


I guess one, by reading section 7 and the examples, can figure out how to encode the a=fmtp parameter, but I think it would to explicitly define the syntax.



Editorial nits: None

v2.23.0 | Report a Bug | By Email