Re: [codec] Ogg Opus zero-length frames

Jean-Marc Valin <jmvalin@jmvalin.ca> Sat, 24 August 2013 04:11 UTC

Return-Path: <jmvalin@jmvalin.ca>
X-Original-To: codec@ietfa.amsl.com
Delivered-To: codec@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 95A6721F9E3F for <codec@ietfa.amsl.com>; Fri, 23 Aug 2013 21:11:35 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.677
X-Spam-Level:
X-Spam-Status: No, score=-2.677 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, HELO_MISMATCH_ORG=0.611, HOST_MISMATCH_COM=0.311, RCVD_IN_DNSWL_LOW=-1]
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 1iwJ+BsfcPrn for <codec@ietfa.amsl.com>; Fri, 23 Aug 2013 21:11:29 -0700 (PDT)
Received: from smtp.mozilla.org (mx1.corp.phx1.mozilla.com [63.245.216.69]) by ietfa.amsl.com (Postfix) with ESMTP id 52C2C21F9E45 for <codec@ietf.org>; Fri, 23 Aug 2013 21:11:25 -0700 (PDT)
Received: from [192.168.1.15] (modemcable130.97-201-24.mc.videotron.ca [24.201.97.130]) (Authenticated sender: jvalin@mozilla.com) by mx1.mail.corp.phx1.mozilla.com (Postfix) with ESMTPSA id 36341F202C; Fri, 23 Aug 2013 21:11:24 -0700 (PDT)
Message-ID: <5218326B.4010503@jmvalin.ca>
Date: Sat, 24 Aug 2013 00:11:23 -0400
From: Jean-Marc Valin <jmvalin@jmvalin.ca>
User-Agent: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:17.0) Gecko/20130801 Thunderbird/17.0.8
MIME-Version: 1.0
To: "Timothy B. Terriberry" <tterribe@xiph.org>
References: <CAMdZqKEDk4rJeEWr-0-oxHQDiy+Lk5QQei9-b+yrXLSRYs8GhQ@mail.gmail.com> <52156299.6080906@xiph.org>
In-Reply-To: <52156299.6080906@xiph.org>
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 7bit
Cc: codec@ietf.org
Subject: Re: [codec] Ogg Opus zero-length frames
X-BeenThere: codec@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Codec WG <codec.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/codec>, <mailto:codec-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/codec>
List-Post: <mailto:codec@ietf.org>
List-Help: <mailto:codec-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/codec>, <mailto:codec-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 24 Aug 2013 04:11:35 -0000

I'm fine with the proposed text in general. I only have two comments below:

On 08/21/2013 09:00 PM, Timothy B. Terriberry wrote:
>    When possible, creating the TOC byte using the same
>    mode, audio bandwidth, channel count, and frame size as the previous
>    packet (if any) covers all losses that do not include a configuration
>    switch, as defined in Section 4.5 of [RFC6716].

Any way you can make that sentence easier to parse?

>    If
>    there is no previous packet, reasonable decoders will not emit
>    anything other than silence regardless of the mode.  Using the CELT-
>    only mode for this case (with any audio bandwidth) allows maximum
>    flexibility, since a single packet can represent any duration up to
>    120 ms that is a multiple of 2.5 ms using at most two bytes.

I think both these sentences should go since they add more confusion
than they help.

	Jean-Marc