Re: [MMUSIC] FW: New Version Notification for draft-ietf-mmusic-rfc4566bis-13.txt

Hi Paul,

-----Original Message-----
From: Paul Kyzivat <pkyzivat@alum.mit.edu>
Date: Tuesday, January 13, 2015 at 7:07 PM
To: "mmusic@ietf.org" <mmusic@ietf.org>
Subject: Re: [MMUSIC] FW: New Version Notification for
draft-ietf-mmusic-rfc4566bis-13.txt

>On 1/13/15 6:10 AM, Ali C. Begen (abegen) wrote:
>> This version addresses Colin’s and Christer’s comments (except
>>Christer’s
>> comment on O/A procedures, which is pending a resolution).
>>
>> Things to do at this point:
>> 1) issues in the tracker
>> http://trac.tools.ietf.org/wg/mmusic/trac/report/7
>>
>> 2) inserting the text from 4566bis-iana-updates
>>
>> If there are things I omitted, please speak up.
>
>Here are some corrections to what appeared in -13:
>
>* Section 6.1:
>
>s/none-ws-string/non-ws-string/

Good catch.

>
>* Section 6.6:
>
>This now says:
>
>       payload-type = integer
>...
>    Note that the payload type number is indicated in a
>    7-bit field, limiting the values to incusively between 0 and 127.
>
>But the definition of 'integer' doesn't include 0.
>
>IMO it was a mistake to define integer that way, but its too late to
>change that. I suggest we define another rule that give a name to a
>number that includes zero. E.g.:
>
>    zero-based-integer = "0" / integer
>
>And then use that here and other places where it is appropriate.

Done.

>
>It could be defined as (1*DIGIT), but that opens the possibility of
>arbitrarily long padding with zeros on the left, so I think the above is
>preferable.
>
>* Section 6.11:
>
>includes:
>
>       sdplang-value = Language-Tag (as defined in <xref
>         target="RFC5646"/>)
>
>The xref didn't get expanded (because it is in artwork.) In any case I
>suggest one of the following:
>
>       sdplang-value = Language-Tag
>       Language-Tag = <defined in RFC5646>
>
>or
>
>       sdplang-value = Language-Tag
>       ; Language-Tag defined in RFC5646

Fixed.

>
>* Section 6.12:
>
>This has the same problem with Language-Tag, but needs a different
>solution since it is a reuse. I thought it was fine in -12.

Used the same solution from the above.

I will address the comments below separately.

-acbegen

>
>* Section 8.2.4:
>
>I see a number of things have been removed from this section. IIUC this
>is because of concerns that came up with Christer tried to follow the
>new style for a draft. The problem there was that all this information
>was included in the IANA Considerations section, and was redundant with
>information in the body of the draft.
>
>But with the new changes there is no longer a requirement to provide
>this information at all. Note that the beginning of this section says:
>
>    New attribute registrations are accepted according to the
>    "Specification Required" policy of [RFC5226], provided that the
>    specification includes the following information:
>
>This calls for the information in the *specification*, not in the IANA
>Considerations section of the specification.
>
>So "Attribute value" needs to be reinstated. This is the only place
>where the syntax is required.
>
>(I'm fine with Long-form going away. It doesn't seem to serve any
>purpose.)
>
>	Thanks,
>	Paul
>
>> -acbegen
>>
>> -----Original Message-----
>> From: "internet-drafts@ietf.org" <internet-drafts@ietf.org>
>> Date: Tuesday, January 13, 2015 at 1:06 PM
>> To: Van Jacobson <van@parc.com>, "Mark J. Handley"
>> <m.handley@cs.ucl.ac.uk>, "Ali C. Begen" <abegen@cisco.com>, Colin
>>Perkins
>> <csp@csperkins.org>, Colin Perkins <csp@csperkins.org>, Van Jacobson
>> <van@parc.com>, "Ali C. Begen" <abegen@cisco.com>, Mark Handley
>> <m.handley@cs.ucl.ac.uk>
>> Subject: New Version Notification for
>>draft-ietf-mmusic-rfc4566bis-13.txt
>>
>>>
>>> A new version of I-D, draft-ietf-mmusic-rfc4566bis-13.txt
>>> has been successfully submitted by Ali Begen and posted to the
>>> IETF repository.
>>>
>>> Name:		draft-ietf-mmusic-rfc4566bis
>>> Revision:	13
>>> Title:		SDP: Session Description Protocol
>>> Document date:	2015-01-13
>>> Group:		mmusic
>>> Pages:		57
>>> URL:
>>> http://www.ietf.org/internet-drafts/draft-ietf-mmusic-rfc4566bis-13.txt
>>> Status:
>>> https://datatracker.ietf.org/doc/draft-ietf-mmusic-rfc4566bis/
>>> Htmlized:      
>>>http://tools.ietf.org/html/draft-ietf-mmusic-rfc4566bis-13
>>> Diff:
>>> http://www.ietf.org/rfcdiff?url2=draft-ietf-mmusic-rfc4566bis-13
>>>
>>> Abstract:
>>>    This memo defines the Session Description Protocol (SDP).  SDP is
>>>    intended for describing multimedia sessions for the purposes of
>>>    session announcement, session invitation, and other forms of
>>>    multimedia session initiation.  This document obsoletes RFC 4566.
>>>
>>>
>>>
>>>
>>>
>>> Please note that it may take a couple of minutes from the time of
>>> submission
>>> until the htmlized version and diff are available at tools.ietf.org.
>>>
>>> The IETF Secretariat
>>>
>>>
>>
>>
>> _______________________________________________
>> mmusic mailing list
>> mmusic@ietf.org
>> https://www.ietf.org/mailman/listinfo/mmusic
>>
>
>_______________________________________________
>mmusic mailing list
>mmusic@ietf.org
>https://www.ietf.org/mailman/listinfo/mmusic
>