[FFmpeg-devel] [PATCH] Updated original IFF demuxer code to be 100% standard IFF-compliance

Sebastian Vater cdgs.basty
Tue Apr 20 20:51:24 CEST 2010

While I was debugging my IFF-ANIM stuff I found (re-remembered) some
facts about the IFF standard which weren't applied to the original
libavformat/iff.c code.

Therefore I was allowed to fix it, and here is it, the patch:

It fixes a bug parsing the ANNO chunk not considering padding.

It fixes that the original code was expecting that the BODY chunk is
always the very last one (most files follow this rule, but this isn't
required by the IFF standard and I've seen files on Amiga long time ago,
where the BODY wasn't last chunk, sometimes stuff like ANNO followed
after BODY, esp. with IFF-8SVX and IFF-SMUS).

It fixes assuming that all known chunks have a fixed size (which isn't
required also, they can be smaller and even be larger (newer data added).

So after application of this patch, the IFF reader should be 100%
compliance to the IFF-standard.


Best regards,
                   :-) Basty/CDGS (-:

-------------- next part --------------
A non-text attachment was scrubbed...
Name: iff-compliance.patch
Type: text/x-patch
Size: 6533 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100420/68ecc888/attachment.bin>

More information about the ffmpeg-devel mailing list