[Gen-art] Gen-art telechat review of draft-ietf-codec-opus-14
Elwyn Davies <elwynd@dial.pipex.com> Sun, 03 June 2012 13:39 UTC
Return-Path: <elwynd@dial.pipex.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 477F821F856D for <gen-art@ietfa.amsl.com>; Sun, 3 Jun 2012 06:39:56 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.599
X-Spam-Level:
X-Spam-Status: No, score=-102.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id XY+Npm9aIE4C for <gen-art@ietfa.amsl.com>; Sun, 3 Jun 2012 06:39:55 -0700 (PDT)
Received: from auth.b.painless.aa.net.uk (b.painless.aa.net.uk [IPv6:2001:8b0:0:30::51bb:1e34]) by ietfa.amsl.com (Postfix) with ESMTP id B532F21F853E for <gen-art@ietf.org>; Sun, 3 Jun 2012 06:39:55 -0700 (PDT)
Received: from 153.107.2.81.in-addr.arpa ([81.2.107.153] helo=[81.187.254.250]) by b.painless.aa.net.uk with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.72) (envelope-from <elwynd@dial.pipex.com>) id 1SbB29-0006k4-CQ; Sun, 03 Jun 2012 14:39:53 +0100
From: Elwyn Davies <elwynd@dial.pipex.com>
To: General Area Review Team <gen-art@ietf.org>
Content-Type: text/plain
Date: Sun, 03 Jun 2012 14:39:49 +0100
Message-Id: <1338730789.31554.15774.camel@mightyatom.folly.org.uk>
Mime-Version: 1.0
X-Mailer: Evolution 2.26.3
Content-Transfer-Encoding: 7bit
Cc: draft-ietf-codec-opus.all@tools.ietf.org
Subject: [Gen-art] Gen-art telechat review of draft-ietf-codec-opus-14
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
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: Sun, 03 Jun 2012 13:39:56 -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>. Please wait for direction from your document shepherd or AD before posting a new version of the draft. Document: draft-ietf-codec-opus-14 Reviewer: Elwyn Davies Review Date: 3 June 2012 IETF LC End Date: IESG Telechat date: 7 June 2012 Summary: As far as the textual part of the document is concerned this version addressed my last call comments and apparently adequately describes the structure of the codec. My review at last call was concentrated on this textual part and I did not have time to significantly dip into the large body of code that makes up the normative part of the draft. I cannot therefore say that there is an exact correspondence between the description and the code, but the description does seem to give a good view (significantly better then before Last Call IMO) of the way the codec operates. I would like to thank the authors for the speedy and effective way that they addressed the comments that I made - to an uninformed neophyte in codecs this seems to be an excellent piece of work. Having had a little time to dip into code since doing the last call, the one serious comment that I would make is that it is a great pity that the code did not incorporate Doxygen or similar code documentation comments. Being able to extract a Doxygen comment document would significantly enhance the understandability of some very complex code and potentially allow the code to be tied back to the textual comments in a way that would (I suspect) help future implementors trying to generate alternative versions. I appreciate that is a lot of work at this late stage but it would help a lot. Regards, Elwyn Davies
- [Gen-art] Gen-art telechat review of draft-ietf-c… Elwyn Davies