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

Reimar Döffinger Reimar.Doeffinger at gmx.de
Wed Sep 18 22:47:20 CEST 2013


On Wed, Sep 18, 2013 at 10:40:55PM +0200, Michael Niedermayer wrote:
> > @@ -720,6 +720,7 @@ typedef struct RcOverride{
> >  
> >  #define CODEC_FLAG2_CHUNKS        0x00008000 ///< Input bitstream might be truncated at a packet boundaries instead of only at frame boundaries.
> >  #define CODEC_FLAG2_SHOW_ALL      0x00400000 ///< Show all frames before the first keyframe
> > +#define CODEC_FLAG2_SKIP_ALPHA    0x00800000 ///< Skip processing alpha if supported by codec
> 
> I suggest to use a field (int or whatever) in the context instead
> of a flag. It reduces the chance of conflicts with additions from
> forks

I went that way because I think we already have the problem with
show_all, kind of thinking one more doesn't make the difference.
But should this happen to be the only objection I can change it.


More information about the ffmpeg-devel mailing list