[FFmpeg-devel] [PATCH] IFF: Add the HAM stuff

Sebastian Vater cdgs.basty
Sun May 16 02:02:59 CEST 2010

Ronald S. Bultje a ?crit :
> Hi,
> On Sat, May 15, 2010 at 7:03 PM, Stefano Sabatini
> <stefano.sabatini-lala at poste.it> wrote:
>> Patch looks fine to me, I suppose it has been tested and works, if no
>> one has more comments I'll apply it in few days.
> Please don't, I'm still reviewing.

Please could you both review this one?

Since we're breaking the compatibility anyway, I just decided to adapt
the IFF extra context not only for packets but for avctx->extradata, too.

So it's possible for the demuxer to pass some flags to decode_init, too.

Both avpkt->data and avctx->extradata are now using the same structure.

So we can pass information to decode_init as well as to each packet. ;-)

Please note that the IFF demuxer now passes the 9 bytes to decode_init
only. The decoder stores it locally.

During packet transmission, the IFF demuxer will just sent a 2 byte
header along with the packets (with a length of 2).

This shows the decoder that the stuff from the previous frame hasn't
changed yet.

Later, when IFF ANIM bumps in the actual packet handling will come to
its full strength. :)


Best regards,
                   :-) Basty/CDGS (-

-------------- next part --------------
A non-text attachment was scrubbed...
Name: iff-ham-support.patch
Type: text/x-patch
Size: 22804 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100516/2a17c96d/attachment.bin>

More information about the ffmpeg-devel mailing list