[Gen-art] Genart Telechat review of draft-ietf-payload-vp8-16.txt

Elwyn Davies <elwynd@dial.pipex.com> Fri, 11 September 2015 19:15 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 (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2543F1B46F9; Fri, 11 Sep 2015 12:15:42 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -101.3
X-Spam-Level:
X-Spam-Status: No, score=-101.3 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, J_CHICKENPOX_26=0.6, USER_IN_WHITELIST=-100] 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 vsneR2Kzr5JD; Fri, 11 Sep 2015 12:15:40 -0700 (PDT)
Received: from auth.a.painless.aa.net.uk (a.painless.aa.net.uk [IPv6:2001:8b0:0:30::51bb:1e33]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D22361B442E; Fri, 11 Sep 2015 12:15:39 -0700 (PDT)
Received: from 1.f.8.5.3.6.b.e.8.e.2.d.5.e.5.a.1.0.0.0.f.b.0.0.0.b.8.0.1.0.0.2.ip6.arpa ([2001:8b0:bf:1:a5e5:d2e8:eb63:58f1]) by a.painless.aa.net.uk with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.77) (envelope-from <elwynd@dial.pipex.com>) id 1ZaTnL-0004rI-4o; Fri, 11 Sep 2015 20:15:37 +0100
To: General area reviewing team <gen-art@ietf.org>, draft-ietf-payload-vp8.all@ietf.org
From: Elwyn Davies <elwynd@dial.pipex.com>
Message-ID: <55F32857.2040200@dial.pipex.com>
Date: Fri, 11 Sep 2015 20:15:35 +0100
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 7bit
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/-Nds4BzcG4g8UphI6woJJsS6i4U>
Subject: [Gen-art] Genart Telechat review of draft-ietf-payload-vp8-16.txt
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: <https://mailarchive.ietf.org/arch/browse/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: Fri, 11 Sep 2015 19:15:42 -0000

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 wait for direction from your
document shepherd or AD before posting a new version of the draft.

For more information, please see the FAQ at

<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document: draft-ietf-payload-vp8-17.txt
Reviewer: Elwyn Davies
Review Date: 2015/09/11
IETF 2nd LC End Date: 2015/07/13
IESG Telechat date: 2015/09/17

Summary: Ready with minor nits.  Thanks for addressing nearly all my 
comments from the 2nd LC review.  The remaining two points are noted 
below. THere is also a point made elsewhere about the Media Type 
Definition.

Major issues:
None.

Minor issues:
s4.2: The consequences of not starting PictureID and/or TL0PICIDX at 
random values (and/or the rationale for starting at random values) is 
not discussed.

Nits/editorial comments:
s4.2: I still consider it undesirable to have two pairs of bit fields 
with the same name (X and M).  Flagging that they are duplicates helps a 
bit but where the values are referred to elsewhere ambiguity remains 
about which of the fields is implied.

s6.1:
>        These parameters are used to signal the capabilities of a receiver
>        implementation.  If the implementation is willing to receive
>        media, both parameters MUST be provided.  These parameters MUST
>        NOT be used for any other purpose.
The latter two sentences would be better placed in s.6.