[codec] Genart last call review of draft-ietf-codec-opus-update-08

Robert Sparks <rjsparks@nostrum.com> Tue, 01 August 2017 18:07 UTC

Return-Path: <rjsparks@nostrum.com>
X-Original-To: codec@ietf.org
Delivered-To: codec@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 47C051321CD; Tue, 1 Aug 2017 11:07:02 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Robert Sparks <rjsparks@nostrum.com>
To: <gen-art@ietf.org>
Cc: codec@ietf.org, ietf@ietf.org, draft-ietf-codec-opus-update.all@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.58.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <150161082225.9505.16606445945798880852@ietfa.amsl.com>
Date: Tue, 01 Aug 2017 11:07:02 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/codec/HGfQfOUhQfazpXSzuykNkvZjYuM>
Subject: [codec] Genart last call review of draft-ietf-codec-opus-update-08
X-BeenThere: codec@ietf.org
X-Mailman-Version: 2.1.22
List-Id: Codec WG <codec.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/codec>, <mailto:codec-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/codec/>
List-Post: <mailto:codec@ietf.org>
List-Help: <mailto:codec-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/codec>, <mailto:codec-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 01 Aug 2017 18:07:02 -0000

Reviewer: Robert Sparks
Review result: Ready

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-codec-opus-update-08
Reviewer: Robert Sparks
Review Date: 2017-08-01
IETF LC End Date: 2017-08-09
IESG Telechat date: Not scheduled for a telechat

Summary: Ready for publication as a Standards Track RFC

This document is straightforward in the changes it is making to OPUS.

My only note of sadness is that it continues to use a documentation mechanism
started by RFC6716 of effectively making a normative reference to the
_proceedings_ of previous IETF meetings. (Note that this document does this
twice: once for the patch file, which is a convenience - the information is in
the draft, and once for the updated test vectors. This is _not_ a convenience,
the information is not in the draft. If, for whatever reason, the proceedings
URL could not be retrieved, someone could not verify their implementation with
the updated test vectors).

On the one hand, we've set the precedent, and we could agree to just let this
go (I'm recommending that to the IESG with this review). On the other hand, we
could make things _slightly_ better (or perhaps just different) by putting the
test vectors in the doc as an appendix as a uuencoded compressed tarball.