IETF Logo Mail Archive

[Gen-art] 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: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 945961AD244 for <gen-art@ietfa.amsl.com>; Sat, 11 Apr 2015 06:23:12 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.6
X-Spam-Level:
X-Spam-Status: No, score=-3.6 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, J_CHICKENPOX_14=0.6, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001] autolearn=ham
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 gIhV6Ng0HaJk for <gen-art@ietfa.amsl.com>; Sat, 11 Apr 2015 06:23:10 -0700 (PDT)
Received: from sesbmg23.ericsson.net (sesbmg23.ericsson.net [193.180.251.37]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 00DCE1AD17F for <gen-art@ietf.org>; Sat, 11 Apr 2015 06:23:09 -0700 (PDT)
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=
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/7HCgjFWJSc6jCdcG5vjf1RwvfYE>
Cc: "draft-ietf-payload-rtp-opus.all@tools.ietf.org" <draft-ietf-payload-rtp-opus.all@tools.ietf.org>
Subject: [Gen-art] Gen-ART review of draft-ietf-payload-rtp-opus-08
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 11 Apr 2015 13:23:12 -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