[FFmpeg-devel] [PATCH] Make decoding alpha optional for some codecs.

wm4 nfxjfg at googlemail.com
Wed Sep 18 20:51:33 CEST 2013


On Wed, 18 Sep 2013 12:24:55 +0200
Michael Niedermayer <michaelni at gmx.at> wrote:

> to keep the code simple tha format could be left as a YUVA format
> only skiping the code that fills the alpha plane
> this should avoid the messy pixfmt setup code

That sounds even worse, because now the alpha plane may contain
garbage, depending on the user options. Why not set the correct pixel
format in the first place?

(Not sure why reimar's patch calls ff_get_format_alpha() everywhere,
instead of doing the reverse at a single point. It would require
adding a ff_get_format_no_alpha() instead. This should be in libavutil,
and use the pixfmt data, btw.)


More information about the ffmpeg-devel mailing list