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

Byron Campen <docfaraday@gmail.com> Thu, 08 October 2015 21:36 UTC

Return-Path: <docfaraday@gmail.com>
X-Original-To: mmusic@ietfa.amsl.com
Delivered-To: mmusic@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A53161ACEA0 for <mmusic@ietfa.amsl.com>; Thu, 8 Oct 2015 14:36:16 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.999
X-Spam-Level:
X-Spam-Status: No, score=-1.999 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, SPF_PASS=-0.001] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id O0dGbyRoV5Vh for <mmusic@ietfa.amsl.com>; Thu, 8 Oct 2015 14:36:15 -0700 (PDT)
Received: from mail-wi0-x229.google.com (mail-wi0-x229.google.com [IPv6:2a00:1450:400c:c05::229]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 9F4D41ACECF for <mmusic@ietf.org>; Thu, 8 Oct 2015 14:36:14 -0700 (PDT)
Received: by wicge5 with SMTP id ge5so43789788wic.0 for <mmusic@ietf.org>; Thu, 08 Oct 2015 14:36:13 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=lIdm7gssbsNoxcbhiOkfiSUbKstKMtTn0us/9jsvrzg=; b=OWc8Q/b/If4FYIPLb+fMyNLWnHdzptOpKl0G30XR5hm4TFM8axNaNQ3Y1uCjiRay2o daGGGtlE6btjJ5MIjzfWwJ1Ks54ntxPsMBc98uwaWFSaecwqwWesUlLMy6vheJo1OSx1 bNHuF6rDJDjQL9HnNNoZMQmCBWeyfdkm6Ov1AFmFL32GEBpNTDxaq/4gfhcc/fD9VHNF IsIpqYOI4pN1nOBtZxdQPjRz/lXTMFCLWQs0Jm045q7NKl9WJq3WoR7edMWXfhiOj8cm fz+AnqHU6VnOaLrR6BKjISVaWuXOYanun5hlbOz9HrOviEoMC/jil2eeByyn4VTJf20z Vhdw==
MIME-Version: 1.0
X-Received: by 10.194.235.6 with SMTP id ui6mr11993900wjc.92.1444340173182; Thu, 08 Oct 2015 14:36:13 -0700 (PDT)
Received: by 10.27.54.16 with HTTP; Thu, 8 Oct 2015 14:36:13 -0700 (PDT)
In-Reply-To: <CADyBEEGpXqRhW8bK_dqx9475Sq8QfnfY-bLz5giruoXEjv+DmQ@mail.gmail.com>
References: <5604B29B.9070005@nostrum.com> <CAMRcRGQngExyrL0doFidea19D8QAvCAgwjoaS=DO+upNGXxL0w@mail.gmail.com> <5612A3C5.8070508@alum.mit.edu> <CADyBEEFjgVR8Yf8RnCii6fuVxf=YN8S6Av4YruwU2PB5Lu_hZg@mail.gmail.com> <CADyBEEGpXqRhW8bK_dqx9475Sq8QfnfY-bLz5giruoXEjv+DmQ@mail.gmail.com>
Date: Thu, 8 Oct 2015 16:36:13 -0500
Message-ID: <CADyBEEE-VD4ELAVLax4N7j_ZLDhB7si68M5aO=YX9Tci+aLbTw@mail.gmail.com>
From: Byron Campen <docfaraday@gmail.com>
To: mmusic@ietf.org
Content-Type: multipart/alternative; boundary=089e01493bd0f522a305219ea682
Archived-At: <http://mailarchive.ietf.org/arch/msg/mmusic/EN9EhJ7dBrf4V1a1Mx0AWj5TxI8>
Subject: Re: [MMUSIC] Comments on draft-pthatcher-mmusic-rid-00
X-BeenThere: mmusic@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Multiparty Multimedia Session Control Working Group <mmusic.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mmusic>, <mailto:mmusic-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mmusic/>
List-Post: <mailto:mmusic@ietf.org>
List-Help: <mailto:mmusic-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mmusic>, <mailto:mmusic-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 08 Oct 2015 21:36:16 -0000

   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 <docfaraday@gmail.com> 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 <docfaraday@gmail.com>
> wrote:
>
>>
>>
>> On Mon, Oct 5, 2015 at 11:22 AM, Paul Kyzivat <pkyzivat@alum.mit.edu>
>> 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
>>
>>
>