Review of draft-ietf-mmusic-4572-update-11

Elwyn Davies <elwynd@dial.pipex.com> Tue, 24 January 2017 20:40 UTC

Return-Path: <elwynd@dial.pipex.com>
X-Original-To: ietf@ietf.org
Delivered-To: ietf@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 272E2129722; Tue, 24 Jan 2017 12:40:35 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Elwyn Davies <elwynd@dial.pipex.com>
To: gen-art@ietf.org
Subject: Review of draft-ietf-mmusic-4572-update-11
X-Test-IDTracker: no
X-IETF-IDTracker: 6.40.4
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <148529043511.12735.9313753556504417778.idtracker@ietfa.amsl.com>
Date: Tue, 24 Jan 2017 12:40:35 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/nPTwciw75I9p76rsM6vxVBQX3Hk>
Cc: draft-ietf-mmusic-4572-update.all@ietf.org, ietf@ietf.org, mmusic@ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.17
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 24 Jan 2017 20:40:35 -0000

Reviewer: Elwyn Davies
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-mmusic-4572-update-11
Reviewer: Elwyn Davies
Review Date: 2017-01-24
IETF LC End Date: 2017-01-26
IESG Telechat date: Not scheduled for a telechat

Summary: Ready.  There are a couple of trivial nits mentioned below. 
Given that the draft has a minimal set of changes apart from the
addition of two sections, I have not read the unchanged pieces in
detail 

Major issues:
None

Minor issues:
None

Nits/editorial comments: 
s1, para 2: s/TLS protocol/The TLS protocol/ (as per RFC 4572)

s4, para 2: s/a new protocol identifier/the protocol identifier/ (it
isn't new any more)

s5.1: Suggest s/m- line/"m" line/  for consistency with s3.4

s5.1, para 1: s/e.g./e.g.,/

s5.1, para 4: s/that each used certificate matches/that each
certificate used matches/

s5.1, para 5: s/each used certificate matches/each certificate used
matches/

s8, para 5: ' This specification creates a new IANA registry named
"Hash Function Textual Names".'  
The registry is no longer new.  Perhaps s/creates a new IANA
registry/takes over the IANA registry from RFC 4572/