[FFmpeg-devel] [PATCH 1/2] pixdesc: mark AV_PIX_FMT_PAL8 as having alpha

Carl Eugen Hoyos cehoyos at ag.or.at
Mon Feb 9 23:36:45 CET 2015


wm4 <nfxjfg <at> googlemail.com> writes:

> Fate still passes.

I am not sure transparency is tested by fate.
Did you test with a pal8 image with transparency?

> -    ((pixdesc)->nb_components == 2 || (pixdesc)->nb_components == 4 
> || (pixdesc)->flags & AV_PIX_FMT_FLAG_PAL)
> +    ((pixdesc)->nb_components == 2 || (pixdesc)->nb_components == 4)

I am not saying this isn't correct (I don't 
know) but isn't this hunk completely 
unrelated to the rest of the patch?

Carl Eugen



More information about the ffmpeg-devel mailing list