Re: [MMUSIC] Gen-ART Last Call review of draft-ietf-mmusic-dtls-sdp-26

[Christer Holmberg <[e-mail]>]

Pull request: https://github.com/cdh4u/draft-dtls-sdp/pull/33

(I kept the example in Section 8, but I fixed the reference)

Regards,

Christer


On 10/07/17 11:08, "Christer Holmberg" <[e-mail]>
wrote:

>Hi Paul,
>
>Thanks for your review! Please see inline.
>
>
>>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
>><​http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
>>
>>Document: draft-ietf-mmusic-dtls-sdp-26
>>Reviewer: Paul Kyzivat
>>Review Date: 2017-07-07
>>IETF LC End Date: 2017-07-24
>>IESG Telechat date: TBD
>>
>>Summary:
>>
>>This draft is basically ready for publication, but has a few nits that
>>should be fixed before publication.
>>
>>Issues:
>>
>>Major: 0
>>Minor: 0
>>Nits:  4
>>
>>(1) NIT:
>>
>>Section 5.3: s/Eventhough/Even though/
>
>Will be fixed.
>
>>(2) NIT:
>>
>>Section 8: s/aTLS/a TLS/
>
>Will be fixed.
>
>>(3) NIT:
>>
>>Section 8: What is the point of including the example? I don't see how
>>it adds anything. Perhaps worked out O/A examples contrasting the
>>differences between the new and existing cases might be marginally
>>helpful. (But IMO not enough to bother with.)
>
>The idea was to take the existing example from RFC 4572 (I note the
>reference is wrong: s/3261/4572), and show how it looks with the tls-id
>attribute.
>
>
>>(4) NIT:
>>
>>Section 10.3.2: s/Througout/throughout/
>
>Will be fixed.
>
>Regards,
>
>Christer
>