Skip to main content

Last Call Review of draft-ietf-mmusic-4572-update-11
review-ietf-mmusic-4572-update-11-genart-lc-davies-2017-01-24-00

Request Review of draft-ietf-mmusic-4572-update
Requested revision No specific revision (document currently at 13)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2017-01-26
Requested 2017-01-12
Authors Jonathan Lennox , Christer Holmberg
I-D last updated 2017-01-24
Completed reviews Opsdir Last Call review of -11 by Tianran Zhou (diff)
Genart Last Call review of -11 by Elwyn B. Davies (diff)
Assignment Reviewer Elwyn B. Davies
State Completed
Request Last Call review on draft-ietf-mmusic-4572-update by General Area Review Team (Gen-ART) Assigned
Reviewed revision 11 (document currently at 13)
Result Ready
Completed 2017-01-24
review-ietf-mmusic-4572-update-11-genart-lc-davies-2017-01-24-00
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/