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

Michael Niedermayer michaelni at gmx.at
Mon Dec 14 18:18:20 CET 2015


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

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

In fact, the RIAA has been known to suggest that students drop out
of college or go to community college in order to be able to afford
settlements. -- The RIAA
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151214/b7724d02/attachment.sig>


More information about the ffmpeg-devel mailing list