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

[Sebastian Vater]

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>