Re: [AVT] I-D ACTION:draft-ietf-avt-rtp-atrac-family-04.txt

[Colin Perkins <csp at csperkins.org>]

On 8 Jul 2005, at 20:50, Internet-Drafts at ietf.org wrote:
> A New Internet-Draft is available from the on-line Internet-Drafts  
> directories.
> This draft is a work item of the Audio/Video Transport Working  
> Group of the IETF.
>
>     Title        : RTP Payload Format for ATRAC Family
>     Author(s)    : M. Romaine, et al.
>     Filename    : draft-ietf-avt-rtp-atrac-family-04.txt
>     Pages        : 28
>     Date        : 2005-7-8
>
> This document describes an RTP payload format for efficient and
>    flexible transporting of audio data encoded with the Adaptive
>    TRansform Audio Codec (ATRAC) family of codecs.  Recent  
> enhancements
>    to the ATRAC family of codecs support high quality audio coding  
> with
>    multiple channels.  The RTP payload format as presented in this
>    document also includes support for data fragmentation, elementary
>    redundancy measures, and a variation on scalable streaming.
>
> A URL for this Internet-Draft is:
> http://www.ietf.org/internet-drafts/draft-ietf-avt-rtp-atrac- 
> family-04.txt

I have a few minor comments on this draft:

 - All figures must be labelled, with a figure number and caption.

 - As I commented on -03, it might make sense for FrgNo to start from 0
   for the first fragment, for consistency with NFrames, rather than
   starting from 1 (this also avoids confusion when wrapping the  
counter,
   since it becomes clear that it wraps to zero, rather than to one).

 - I notice that the "Intended usage" part of the MIME registration has
   changed from "LIMITED USE" to "COMMON", yet the draft still  
indicates
   that an ATRAC license is required. This would seem to imply "LIMITED
   USE" would be appropriate.

Otherwise, this looks good.

Colin



_______________________________________________
Audio/Video Transport Working Group
avt at ietf.org
https://www1.ietf.org/mailman/listinfo/avt