[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