[FFmpeg-devel] [PATCH] Video decoder and demuxer for AMV files

[Vitor Sessak]

Hi

Michael Niedermayer wrote:
> Hi
> 
> On Tue, Sep 25, 2007 at 07:51:12PM +0200, Vitor Sessak wrote:
>> Vladimir Voroshilov wrote:
>>> 2007/9/25, Vitor Sessak <vitor1001 at gmail.com>:
>>>> Actually, I wanted to make explicit the similarity to the smjpeg audio
>>>> decoder. But I found a way of doing it without duplicating code. New
>>>> patch attached.
>>> Page[1] says (and all my samples confirm this), that AMV audio samples
>>> has additional 8 bytes (instead of 4) at the top:
>>> 1-2 predictor
>>> 3-4 index
>>> 5-8 sample size
>>> Thus your patch should be modified to skip additional 4 bytes before
>>> converting nibbles.
>> You're absolutely right (it fixed pops). New patch attached.
> 
> looks ok

commited.

-Vitor