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

Sebastian Vater cdgs.basty
Tue Apr 20 22:56:29 CEST 2010

Hi Ronald!

Ronald S. Bultje a ?crit :
> That's not OK. A file with only width but no height is clearly
> invalid. I doubt they exist.
> Can probably be generalized at the end of the loop (before the switch)
> to decrease lines-of-code.
> That would break things badly, no? Don't skip 1 byte BEFORE reading
> the data. And you're no longer exiting the loop so we will never
> actually decode the file.
Fixed partially.


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

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

More information about the ffmpeg-devel mailing list