[ scummvm-Patches-2971742 ] Scumm: autodetection for diskimage contents

[SourceForge.net]

Patches item #2971742, was opened at 2010-03-17 04:56
Message generated for change (Tracker Item Submitted) made by tobigun
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=2971742&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: Tobias Gunkel (tobigun)
Assigned to: Nobody/Anonymous (nobody)
Summary: Scumm: autodetection for diskimage contents

Initial Comment:
At the moment, ScummVM checks MD5 hashes only for the C64 .d64 or Apple .dsk disk-image containers but not for its contents. This is inconvenient as ScummVM is also able to handle the unpacked LFL game data. So those LFLs are not detected at the moment. In addition ScummVM might not recognize the game correctly if the disk directory was changed or if a savegame was saved on the disk (I remember that some people used floppy hole punchers to make the floppies writeable but I don't know if saving a savegame destroyed the game date).

The patch just calculates MD5 hashes for the 00.LFL files even if just a disk-image is present, so there is no need to add two MD5 hashes for each version. There were five MD5 hashes for the C64 versions before (two English, two German and one Italian). I already added the hashes for my two German versions.

It would be cleaner if the SearchMan.addDirectory() and SearchMan.remove() calls could be removed somehow, but File::open() which is called by ScummDiskImage requires the path of the two disk images to be registered in the SearchManager.

Maybe you like this patch. If not, maybe someone can use this as a base for a better approach.


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

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