[FFmpeg-devel] [PATCH] lavc/utils: use AVPixFmtDescriptor to probe palette formats

Matthieu Bouron matthieu.bouron at gmail.com
Tue Dec 15 10:38:07 CET 2015


On Mon, Dec 14, 2015 at 6:18 PM, Michael Niedermayer <michaelni at gmx.at>
wrote:

> On Mon, Dec 14, 2015 at 05:49:48PM +0100, Matthieu Bouron wrote:
> > From: Matthieu Bouron <matthieu.bouron at stupeflix.com>
> >
> > Also use the input frame format instead of the AVCodecContext one
> according
> > to the documentation of AVCodecContext.get_buffer2().
> > ---
> >
> > The current code rely on the fact that avpriv_set_systematic_pal2
> > will error out if the format is not a palette one (and that the error
> > code is not checked).
> > I find it makes more sense to probe this kind of formats using the
> > AVPixFmtDescriptor flags.
> >
> > Also the documentation of AVCodecContext.get_buffer2() mention that the
> > frame fields must be used instead of the AVCodecContext one.
> >
> > Matthieu
> >
> > ---
> >  libavcodec/utils.c | 13 +++++++++++--
> >  1 file changed, 11 insertions(+), 2 deletions(-)
>
> probably ok
>

Pushed. Thanks.

[...]


More information about the ffmpeg-devel mailing list