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/

* 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.

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

* 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.

* 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
>