[Ffmpeg-devel] snow MC simplification

Ivan Kalvachev ikalvachev
Mon Jan 16 10:23:29 CET 2006


2006/1/16, Michael Niedermayer <michaelni at gmx.at>:
> Hi
>
> On Mon, Jan 16, 2006 at 12:12:53AM +0100, Michael Niedermayer wrote:
> > 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) )
>
> enough fun, patch attched :)

Is this patch for the patch of snow patch or
it is just patch for snow patch?

(sorry, I couldn't resist either:)





More information about the ffmpeg-devel mailing list