[FFmpeg-devel] [PATCH] Issue 579: Theora Thusnelda 4MV uncoded support

Aurelien Jacobs aurel
Mon Sep 1 23:51:26 CEST 2008


On Mon, 01 Sep 2008 19:55:43 +0200
matthieu castet <castet.matthieu at free.fr> wrote:

> Aurelien Jacobs wrote:
> > On Sat, 30 Aug 2008 16:27:14 -0400
> > "Alex Converse" <alex.converse at gmail.com> wrote:
> > 
> >> Hi,
> >>
> >> Attached is an updated version of the patch that does not add trailing
> >> white space, does not search for uncoded on non-4mv blocks and does
> >> not set up an extra vector. Please ignore the previous one and
> >> consider this instead.
> > 
> > This patch still contains various cosmetics...
> > I reworked it. See attached simplified version split in 2 patches.
> > 
> >  - The first one just rearange setting of prior_last_motion_* and
> >    last_motion_*, with no functionnal change. It helps to keep
> >    second patch small and simple.
> >  - The second one now only skips MV reading for blocks which are
> >    not coded. It is much simpler and obvious than your initial
> >    patch.
> > 
> > This is tested to work fine with the sample from issue579 and also
> > with other "traditional" theora files.
> > 
> But does it work with vp3 files ?

Oh, I forgot to precise it, but yes, it works with vp3.

> I didn't check but I think these is on the common theora/vp3 code.

Yes, this code is common to theora and vp3, and it still works
fine for vp3.

Aurel




More information about the ffmpeg-devel mailing list