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

[Carl Eugen Hoyos]

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