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

[Michael Niedermayer]

On Sat, Nov 08, 2008 at 11:59:33AM +1100, Peter Ross wrote:
> 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/ ?

iam fine with both of these

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator
-------------- 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/20081112/94d67b60/attachment.pgp>