[FFmpeg-devel] [PATCH 1/2] lavc: add a channel_layout field to AVFrame.

[Michael Niedermayer]

On Sat, Apr 28, 2012 at 08:04:31PM +0200, Nicolas George wrote:
> The field is filled with the codec context information.
> 
> FIXME need a minor version bump.
> 
> Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
> ---
>  libavcodec/avcodec.h |   10 ++++++++++
>  libavcodec/options.c |    1 +
>  libavcodec/utils.c   |    2 ++
>  3 files changed, 13 insertions(+), 0 deletions(-)
> 
> 
> Thanks for the review.
> 
> I guess libavfilter is "code outside libavcodec", so I have to use that
> av_opt_ptr in the places where I use these fields.

yes

and patch still LGTM

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

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120428/4b792c80/attachment.asc>