[FFmpeg-devel] [PATCH] lavc: add channels field to AVFrame

[Stefano Sabatini]

On date Monday 2012-07-30 21:24:47 +0200, Michael Niedermayer encoded:
> On Mon, Jul 30, 2012 at 07:03:23PM +0200, Stefano Sabatini wrote:
> > On date Monday 2012-07-30 18:49:32 +0200, Michael Niedermayer encoded:
> > > On Mon, Jul 30, 2012 at 04:11:36PM +0200, Stefano Sabatini wrote:
> > > > This is required otherwise it is not always possible to guess the number
> > > > of channels from the layout, for example if the channel layout is
> > > > unknown.
> > > > ---
> > > >  libavcodec/avcodec.h |   11 +++++++++++
> > > >  libavcodec/utils.c   |    4 ++++
> > > >  2 files changed, 15 insertions(+), 0 deletions(-)
> > > 
> > > LGTM
> > > 
> > > thanks
> > 
> > Nit: should be nb_channels (more meaningful, consistent with
> > nb_samples) or channels (consistent with AVCodecContext)?
> 
> whichever name you prefer ...
> i would probably favor consistency to the equivalent field in
> other structs like AVCodecContext but thats only a very slight
> preferance
> 
> 
> > 
> > I'm not still sure it is a good idea to propagate the info to lavfi
> > (since we may require that the channel layout is always set inside the
> > filtergraph), but this patch seems useful on its own and shouldn't
> > hurt anyway.
> 
> yes, especially when the number changes having it in AVFrame is a good
> idea

Applied.
-- 
FFmpeg = Free Fabulous Mythic Programmable Evil God