Hi, On Sat, Jan 28, 2012 at 8:42 AM, Ronald S. Bultje <[email protected]> wrote: > On Sat, Jan 28, 2012 at 12:46 AM, Diego Biurrun <[email protected]> wrote: >> On Fri, Jan 27, 2012 at 07:54:36AM +0800, Ronald S. Bultje wrote: >>> + if (flags & AV_CPU_FLAG_MMX2) { >>> + dsp->add_png_paeth_prediction = ff_add_png_paeth_prediction_mmx2; >>> + } >>> + >>> + if (flags & AV_CPU_FLAG_SSE2) { >>> + dsp->add_bytes_l2 = ff_add_bytes_l2_sse2; >>> + } >>> + >>> + if (flags & AV_CPU_FLAG_SSSE3) { >>> + dsp->add_png_paeth_prediction = ff_add_png_paeth_prediction_ssse3; >>> + } [..] >> Also, I'd drop the pointless {} and make this more concise. > > I forgot doing this, it causes a lot of conflicts when applying each > successive patch.
Fixed now. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
