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

[Michael Niedermayer]

On Wed, Sep 18, 2013 at 08:51:33PM +0200, wm4 wrote:
> 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.)

yes, this is a possibility as well

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

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130918/8be1f848/attachment.asc>