Re: [MMUSIC] Comments on draft-pthatcher-mmusic-rid-00

[Byron Campen <[e-mail]>]

   Yet more feedback; using ';' as a delimiter for rid-fmt-list could lead
to some confusion:

a=rid:1 send pt=96;97;98;max-width=800

   It would probably be good to avoid the use of ';' as a delimiter here
too.

Best regards,
Byron Campen

On Thu, Oct 8, 2015 at 3:52 PM, Byron Campen <[e-mail]> wrote:

>    Some additional feedback on the grammar:
>
> - The use of ';' as a delimiter for rid-list complicates the parse code if
> ';' is also used as a delimiter for rid-attr-list. For example, consider:
>
> a=rid:3 send pt=96 depend=1;2;max-fps=30
> vs
> a=rid:3 send pt=96 depend=1,2;max-fps=30
>
>
> - There does not seem to be anywhere for extensions to rid-attr-list to
> hook in. We should make room for that, and forbid any such extensions from
> using ';' since that is reserved as a delimiter.
>
> - There is an example where max-br takes no value:
>
> a=rid:1 send pt=97; max-br=; max-height=720;
>
>    Is this intended?
>
> Best regards,
> Byron Campen
>
> On Thu, Oct 8, 2015 at 12:47 PM, Byron Campen <[e-mail]>
> wrote:
>
>>
>>
>> On Mon, Oct 5, 2015 at 11:22 AM, Paul Kyzivat <[e-mail]>
>> wrote:
>>
>>> * Section 10.*:
>>>
>>> The a=rid lines in most/all of the examples are inconsistent with the
>>> syntax. E.g. The following:
>>>
>>>    a=rid:1 send pt=*; max-width=1280; max-height=720; max-fps=30
>>>
>>> ought to be:
>>>
>>>    a=rid:1 send pt=* max-width=1280;max-height=720;max-fps=30
>>>
>>>
>>    Actually, it is worse than this; the grammar does not permit more than
>> one of "max-width", "max-height", etc. Also, the use of <param-val> (aka
>> <byte-string>) as the values of things like max-width precludes the use of
>> any delimiter, since param-val can contain anything except NUL, CR, and LF.
>> Things like max-width really should be a positive integer, so the grammar
>> will need to be tightened down. I'm guessing that what the author was
>> shooting for was ';' delimited, with positive integer values for everything
>> but depend=.
>>
>>    Also, I would expect that stuff like pt=*,96,8,* is not permitted, but
>> the current grammar permits it.
>>
>> Best regards,
>> Byron Campen
>>
>>
>