Hi,
On Tue, Feb 21, 2012 at 11:16 AM, Diego Biurrun <[email protected]> wrote:
> case PIX_FMT_RGBA:
> - case PIX_FMT_BGRA: return (CONFIG_SWSCALE_ALPHA && c->srcFormat ==
> PIX_FMT_YUVA420P) ? yuva2rgba_c : yuv2rgb_c_32;
> - case PIX_FMT_RGB24: return yuv2rgb_c_24_rgb;
> - case PIX_FMT_BGR24: return yuv2rgb_c_24_bgr;
> + case PIX_FMT_BGRA:
> + return (CONFIG_SWSCALE_ALPHA && c->srcFormat == PIX_FMT_YUVA420P)
> + ? yuva2rgba_c : yuv2rgb_c_32;
Wrong '?' placement? '?' should go on line before. Also, an if() { }
else { } may be prettier.
Rest is OK.
Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
