Re: [Gen-art] Gen-ART review of draft-ietf-avtext-client-to-mixer-audio-level-05.txt

Alexey Melnikov <alexey.melnikov@isode.com> Mon, 26 September 2011 22:22 UTC

Return-Path: <alexey.melnikov@isode.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id CFEEC11E8087; Mon, 26 Sep 2011 15:22:20 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.551
X-Spam-Level:
X-Spam-Status: No, score=-102.551 tagged_above=-999 required=5 tests=[AWL=0.048, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id PEbgjWsKegxJ; Mon, 26 Sep 2011 15:22:20 -0700 (PDT)
Received: from rufus.isode.com (rufus.isode.com [62.3.217.251]) by ietfa.amsl.com (Postfix) with ESMTP id 1C87F11E80B4; Mon, 26 Sep 2011 15:22:20 -0700 (PDT)
Received: from [188.29.120.97] (188.29.120.97.threembb.co.uk [188.29.120.97]) by rufus.isode.com (submission channel) via TCP with ESMTPA id <ToD7vAAvpT7W@rufus.isode.com>; Mon, 26 Sep 2011 23:25:02 +0100
Message-ID: <4E80FBB8.4070205@isode.com>
Date: Mon, 26 Sep 2011 23:24:56 +0100
From: Alexey Melnikov <alexey.melnikov@isode.com>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.12) Gecko/20050915
X-Accept-Language: en-us, en
To: Jonathan Lennox <jonathan@vidyo.com>
References: <4E7FA0BE.1050201@isode.com> <C3759687E4991243A1A0BD44EAC823034C42868FB9@BE235.mail.lan>
In-Reply-To: <C3759687E4991243A1A0BD44EAC823034C42868FB9@BE235.mail.lan>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; format="flowed"
Content-Transfer-Encoding: 7bit
Cc: General Area Review Team <gen-art@ietf.org>, "draft-ietf-avtext-client-to-mixer-audio-level.all@tools.ietf.org" <draft-ietf-avtext-client-to-mixer-audio-level.all@tools.ietf.org>, "ietf@ietf.org" <ietf@ietf.org>
Subject: Re: [Gen-art] Gen-ART review of draft-ietf-avtext-client-to-mixer-audio-level-05.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 26 Sep 2011 22:22:21 -0000

Jonathan Lennox wrote:

>Hi, Alexey -- thank you for the Gen-ART review.
>  
>
Hi Jonathan,

>Alexey Melnikov writes:
>  
>
>>Question: are the two encoding of the audio level indication option specified in the document really necessary?
>>    
>>
>
>Do you mean the one-byte vs. two-byte forms of the header extension (Figure 1 vs. Figure 2)?  These are the two forms of the generic header extensions defined by RFC 5285.
>
I understood that. Does RFC 5285 require that both forms should be allowed?
In general, it would be good to avoid multiple representations of the 
same thing.

>The actual payload (one byte containing the V and level bits) is identical in the two cases; the only difference is the container.  We can add some text clarifying this point if you think it would be helpful.
>  
>
>>Nits/editorial comments:
>>s/relys/relies ???
>>    
>>
>Thanks, will fix.
>  
>