On Mon, Oct 3, 2011 at 12:18 PM, Kieran Kunhya <kier...@ob-encoder.com> wrote: > > On Mon, Oct 3, 2011 at 11:17 AM, Ronald S. Bultje <rsbul...@gmail.com> wrote: >> >> Hi, >> >> On Fri, Sep 30, 2011 at 6:11 PM, Kieran Kunhya <kier...@ob-encoder.com> >> wrote: >> > /me blames gmail >> >> Nice stuff, mostly OK. Small comments: >> >> > + yuv2planarX_fn yuv2plane_luma; >> > + yuv2planarX_fn yuv2plane_chroma; >> > + yuv2interleavedX_fn yuv2nv12X; >> >> To remain consistent with the X/1 suffix model, I'd call the first two >> yuv2planeX_luma/chroma. The last one only handles chroma and you could >> thus volunteer to call it yuv2nv12X_chroma. >> >> Shouldn't the patch comment out the SIMD yuv2yuvX functions (x86 and ppc)? >> >> Ronald > > Patch 1 comments out the simd x86 ones but I forgot about ppc. I will address > the naming.
Patch updated.
0001-Split-up-yuv2yuvX-functions.patch
Description: Binary data
_______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel