[Ffmpeg-cvslog] r5794 - trunk/libavcodec/vp3.c

[Mike Melanson]

Michael Niedermayer wrote:
> Hi
> 
> On Wed, Jul 19, 2006 at 03:31:38PM -0700, Mike Melanson wrote:
>> Michael Niedermayer wrote:
>>> no, theres plenty left
>> I can't wait to find out if we can finally make it match up to the Win32
>> binary for raw performance.
> 
> we certainly can, but optimizing vp3 is not anywhere on my todo list
> iam just trying to cleanup some of the mess there is in vp3.c and
> relatives, the code could very well become a little faster as a
> sideeffect but thats not my primary goal
> 
> and anyway, code should be cleaned up before it is optimized otherwise
> you could end up writing asm code for a page of complex c code which 
> does nothing but i++

Always a good idea. There was a bunch of code in that module that I
wanted to simplify once I was sure everything worked correctly. But it
took so long to get to that point, as you know. :)

I'm glad you brought the prediction logic down to size. I worked hard
trying to find a better way to write that, mostly because I didn't just
want to replicate the original On2 code. In the end, that was the only
method I could find.

-- 
	-Mike Melanson