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

[Michael Niedermayer]

Hi

On Tue, Sep 25, 2007 at 07:51:12PM +0200, Vitor Sessak wrote:
> Hi
>
> Vladimir Voroshilov wrote:
>> Hi, Vitor
>> 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

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070927/072f5836/attachment.pgp>