[ scummvm-Patches-2122869 ] ALL: Common load dialog

[SourceForge.net]

Patches item #2122869, was opened at 2008-09-22 16:49
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=2122869&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: Filippos Karapetis (thebluegr)
Assigned to: Nobody/Anonymous (nobody)
Summary: ALL: Common load dialog

Initial Comment:
Hello

This is a first attempt at creating a common load dialog for all ScummVM engines.

It adds a new virtual class to Engine(), loadGameState(), which accepts a game slot and calls the corresponding engine's load state method. I've added some test code for the SAGA engine. The igor, queen and touche engines should support this correctly without any major changes (they already have a loadGameState method). I haven't modified the other engines in this patch.

I'm unsure if a new engine feature should be added, (e.g. kSupportsLoading) - it seems that engines that support kSupportsDirectLoad should be easy to change.

Currently, I've reused the code for the load dialog from the game launcher, though this could be moved to engines/dialogs.*

It should be pretty straightforward to add a common save dialog as well (though I haven't done it in this patch).

Thoughts/opinions are welcome :)

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

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