On Tue, Feb 21, 2012 at 11:39:40AM -0800, Ronald S. Bultje wrote: > 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.
Replaced by if-else in upcoming patch. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
