Hi,
On Mon, May 2, 2016 at 5:49 AM, Carl Eugen Hoyos <ceho...@ag.or.at> wrote:
> On Wednesday 13 April 2016 10:27:03 am you wrote:
> > Maybe it should just check whether the pixel format is alpha, instead
> > of whether it's not yuv420p. (Just a suggestion.)
>
> Isn't that exactly equivalent to what my patch is doing?
> Or do I just misunderstand?
>
> I just thought that "if (pix_fmt != AV_PIX_FMT_YUV420P)" is shorter than
> "if (pix_fmt == AV_PIX_FMT_RGB32 || pix_fmt == AV_PIX_FMT_YUVA420P)". Do
> you
> disagree?
I think he means:
AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(pix_fmt);
if (desc->flags & AV_PIX_FMT_FLAG_ALPHA) {
flags |= 16;
}
Ronald
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
