Opened 12 years ago

Closed 12 years ago

Last modified 2 years ago

#8921 closed patch

Launcher/MetaEngine: Finnished removeSaveState

Reported by: lordhoto Owned by: lordhoto
Priority: normal Component: --Other--
Keywords: Cc:
Game:

Description

Hi,

this patch makes the 'removeSaveState' method in MetaEngine usable for the launcher load/delete dialog. To achieve this I introduced a 'target' parameter for 'removeSaveState' also, so the plugin implementation is able to create the correct savefile name with it.

I implemented the new 'removeSaveState' for all engines flagged with 'kSupportsDeleteSave'.

Next I changed the Launcher implementation of deleting save states to use MetaEngine::removeSaveState instead of using SaveFileManager directly.

I wasn't able to test all engines, but it *should* work. (Actually I even have Touche disabled in my build, same for queen, so if it doesn't compile I made some stupied typo :-)

Ticket imported from: #2105593. Ticket imported from: patches/1026.

Attachments (1)

saveStateRem_v1.patch (11.7 KB ) - added by lordhoto 12 years ago.
Patch against r34490.

Download all attachments as: .zip

Change History (4)

by lordhoto, 12 years ago

Attachment: saveStateRem_v1.patch added

Patch against r34490.

comment:1 by lordhoto, 12 years ago

Owner: changed from fingolfin to lordhoto
Status: newclosed

comment:2 by lordhoto, 12 years ago

Committed my patch now, I didn't came across any problems yet. And if it should be easy to fix.

comment:3 by digitall, 2 years ago

Component: --Other--
Note: See TracTickets for help on using tickets.