On Thu, May 26, 2011 at 9:53 PM, Diego Biurrun <[email protected]> wrote:
> On Thu, May 26, 2011 at 02:56:57PM -0400, Daniel Kang wrote: > > Updated patch with Loren's (thanks Loren) comments, faster functions, and > > better second commit message. > > > From 48b7c51842e95402fc11406abe0d272264acab8f Mon Sep 17 00:00:00 2001 > > From: Daniel Kang <[email protected]> > > Date: Tue, 24 May 2011 15:14:38 -0400 > > Subject: [PATCH 1/2] Add IDCT functions for 10-bit H.264. > > > > Ports the majority of IDCT functions for 10-bit H.264. > > > > Some code was ported from x264. > > This is confusing - where was the rest ported from if not x264? > I suspect it is just x264, so I would push tomorrow with the > following log message: > > x86: Add IDCT assembler functions for 10-bit H.264. > > This ports the majority of IDCT functions for 10-bit H.264 from x264. > Only some parts were ported from x264. Some were also ported from the 8-bit H.264 code in libav. Should I clarify which were from which? Loren gave me a few more comments, which I will address tomorrow. This patch also increases overall decoding speed by 10% on the clip I tested.
_______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
