[ scummvm-Patches-2704062 ] New IFF parser

SourceForge.net noreply at sourceforge.net
Sun Mar 22 21:03:30 CET 2009


Patches item #2704062, was opened at 2009-03-22 21:03
Message generated for change (Tracker Item Submitted) made by peres001
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=2704062&group_id=37116

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: peres (peres001)
Assigned to: Nobody/Anonymous (nobody)
Summary: New IFF parser

Initial Comment:
I was extending the ILBM format decoder for Big Red Adventure when I ran into a better implementation of the underlying IFF parser (in Kyra).

The current IFF parser only allows sequential access to the chunks, whereas the one in the patch doesn't have this limitation. As a result, client code can effectively search for the chunks it needs instead of scanning all the file accumulating state.

The ILBM decoder I was working on came out so much better without switch()'s, that I ended up ditching the whole current IFFParser and adapted its users: ILBMDecoder, PBMDecoder and A8SVXDecoder.

The patch contains the related changes, together with adaptations to Parallaction (for A8SVXDecoder), and a one-liner in Saga (for PBMDecoder).

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=2704062&group_id=37116




More information about the Scummvm-tracker mailing list