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 DE8341B2CB4
 for <mmusic@ietfa.amsl.com>; Thu,  8 Oct 2015 10:47:10 -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 fuX59UoKAE1Y for <mmusic@ietfa.amsl.com>;
 Thu,  8 Oct 2015 10:47:09 -0700 (PDT)
Received: from mail-wi0-x22a.google.com (mail-wi0-x22a.google.com
 [IPv6:2a00:1450:400c:c05::22a])
 (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 334581B2CB3
 for <mmusic@ietf.org>; Thu,  8 Oct 2015 10:47:09 -0700 (PDT)
Received: by wiclk2 with SMTP id lk2so39262859wic.0
 for <mmusic@ietf.org>; Thu, 08 Oct 2015 10:47:07 -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
 :cc:content-type;
 bh=m/2YpThuMNhfkwXhWkz2JBPmeApUcpHfznjrI+aMgVA=;
 b=LTi4rgM0rN5nVVyqIJXTCmmPBKzJggotWfaSpuZkRhP2JbEb+xZW1YQ77Hjaqg3iQf
 UcdFPujrUoYDG4EkmmMNi1nH83/hG6Xkz8ySQSJl2HC5VKi4TVeIykhuazdrZYonDEx5
 Q1vAnJkGsR/jq6W0xv3GsnhXu6pmEfDUZElZXguukQ4tcm7yyneFrgwAZRsg4eft7Ium
 jmNw9P6CNeXrfUZcW4jHcKeTBEQoMxxYF+dgheLUtsKouW2zF19YPPTEXwGHRef7hMMz
 nqVElLtjto08osGsXgM8Ri1rJCQgGtEeAjvFJ94S6bbh/SBzlw4trZGmIJAe7IMp0IrZ
 Xy/g==
MIME-Version: 1.0
X-Received: by 10.194.134.197 with SMTP id pm5mr9509640wjb.118.1444326427725; 
 Thu, 08 Oct 2015 10:47:07 -0700 (PDT)
Received: by 10.27.54.16 with HTTP; Thu, 8 Oct 2015 10:47:07 -0700 (PDT)
In-Reply-To: <5612A3C5.8070508@alum.mit.edu>
References: <5604B29B.9070005@nostrum.com>
 <CAMRcRGQngExyrL0doFidea19D8QAvCAgwjoaS=DO+upNGXxL0w@mail.gmail.com>
 <5612A3C5.8070508@alum.mit.edu>
Date: Thu, 8 Oct 2015 12:47:07 -0500
Message-ID: <CADyBEEFjgVR8Yf8RnCii6fuVxf=YN8S6Av4YruwU2PB5Lu_hZg@mail.gmail.com>
From: Byron Campen <docfaraday@gmail.com>
To: Paul Kyzivat <pkyzivat@alum.mit.edu>
Content-Type: multipart/alternative; boundary=089e01227dc2aa19a405219b7343
Archived-At: <http://mailarchive.ietf.org/arch/msg/mmusic/tVsN2YsQkCQFkRwAjbyUv5EqWTM>
Cc: mmusic@ietf.org
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 17:47:11 -0000

--089e01227dc2aa19a405219b7343
Content-Type: text/plain; charset=UTF-8

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

--089e01227dc2aa19a405219b7343
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><br><div class=3D"gmail_extra"><br><div class=3D"gmail_quo=
te">On Mon, Oct 5, 2015 at 11:22 AM, Paul Kyzivat <span dir=3D"ltr">&lt;<a =
href=3D"mailto:pkyzivat@alum.mit.edu" target=3D"_blank">pkyzivat@alum.mit.e=
du</a>&gt;</span> wrote:<br><blockquote class=3D"gmail_quote" style=3D"marg=
in:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1e=
x">
* Section 10.*:<br>
<br>
The a=3Drid lines in most/all of the examples are inconsistent with the syn=
tax. E.g. The following:<br>
<br>
=C2=A0 =C2=A0a=3Drid:1 send pt=3D*; max-width=3D1280; max-height=3D720; max=
-fps=3D30<br>
<br>
ought to be:<br>
<br>
=C2=A0 =C2=A0a=3Drid:1 send pt=3D* max-width=3D1280;max-height=3D720;max-fp=
s=3D30<br>
<br></blockquote><div><br></div><div>=C2=A0=C2=A0 Actually, it is worse tha=
n this; the grammar does not permit more than one of &quot;max-width&quot;,=
 &quot;max-height&quot;, etc. Also, the use of &lt;param-val&gt; (aka &lt;b=
yte-string&gt;) 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&#39;m guessing that what the author was =
shooting for was &#39;;&#39; delimited, with positive integer values for ev=
erything but depend=3D.<br><br></div><div>=C2=A0=C2=A0 Also, I would expect=
 that stuff like pt=3D*,96,8,* is not permitted, but the current grammar pe=
rmits it.<br></div><div><br></div><div>Best regards,<br></div><div>Byron Ca=
mpen<br></div><div><br></div></div></div></div>

--089e01227dc2aa19a405219b7343--

