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

[Aurelien Jacobs]

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.

As this is quite obvious, I plan to apply this soon.
OK Mike ? (you're the maintainer IIRC)

Aurel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: theora-4mv-uncoded_1.diff
Type: text/x-diff
Size: 1675 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080901/d2d0b2b0/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: theora-4mv-uncoded_2.diff
Type: text/x-diff
Size: 1574 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080901/d2d0b2b0/attachment-0001.diff>