[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.
>
Fixed.
> Can probably be generalized at the end of the loop (before the switch)
> to decrease lines-of-code.
>
Fixed.
> 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