[Ffmpeg-devel] snow MC simplification

Michael Niedermayer michaelni
Mon Jan 16 00:12:53 CET 2006


Hi

On Sun, Jan 15, 2006 at 09:41:54PM +0100, Guillaume POIRIER wrote:
> Hi,
> 
> On 1/8/06, Michael Niedermayer <michaelni at gmx.at> wrote:
> > Hi
> >
> > Snow currently uses the h.264 luma MC functions if possible and if not
> > (1/8 pel) then its own MC functions (mc_block())
> > the result is that mc_block() is only used for some chroma blocks if qpel is
> > enabled, so it would be nice if we could et rid of it, the attached patch
> > does that, but it needs testing (encode stuff with and without and view with
> > the respective player with and without the patch)
> 
> File to patch: snow.c
> patching file snow.c
> Hunk #1 succeeded at 2509 (offset 11 lines).
> Hunk #2 FAILED at 2529.
> 1 out of 2 hunks FAILED -- saving rejects to file snow.c.rej
> 
> rejected!
> 
> (sorry, I couldn't resist! ;o) )
> 
> Now, seriously, it seems this code doesn't apply anymore (even by
> hand), and I haven't had a chance to test it until now.
> 
> Michael, can you send an updated patch?

yes i can

(sorry, I couldn't resist either! ;o) )

[...]
-- 
Michael





More information about the ffmpeg-devel mailing list