[FFmpeg-devel] [PATCH] Channel layouts API doxygen

Peter Ross pross
Sat Nov 8 01:59:33 CET 2008


On Sat, Nov 08, 2008 at 01:21:31AM +0100, Michael Niedermayer wrote:
> On Sat, Nov 08, 2008 at 11:08:17AM +1100, Peter Ross wrote:
> > On Thu, Nov 06, 2008 at 03:02:02PM +0100, Michael Niedermayer wrote:
> > > On Wed, Nov 05, 2008 at 10:04:42PM +1100, Peter Ross wrote:
> > > > 
> > > > This patch amends the AVCodec.channel_layouts comment to clarify
> > > > concerns about the relevance of this field for decoders.
> > > 
> > > Its becoming a little long, maybe a syntax similar to AVCodecContext would
> > > be clearer ...
> > > 
> > > [...]
> > 
> > Hows about this then.
> [...]
> > +     * Array of supported channel layouts, or NULL if unknown. Array is terminated by 0.
> > +     * encoder - Set by libavcodec.
> 
> iam not sure if saying "set by lavc" is really correct, lavc isnt really
> setting it, actually a user provided AVCodec could be used in which case
> its actually the user setting it, lavc wont ever write to it ...

Suggestions:

s/Set by libavcodec/mandatory/ ?
s/Set by libavcodec/required/ ?
s/Set by libavcodec/usued/ ?

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081108/b939142e/attachment.pgp>



More information about the ffmpeg-devel mailing list