Opened 5 months ago

Closed 5 months ago

#12845 closed defect (fixed)

GUI: AUTOSAVE OPTION: Enabling Autosave displays warning when no non-autosave saved games exist.

Reported by: macca8 Owned by: macca8
Priority: normal Component: GUI
Version: Keywords:
Cc: Game:

Description (last modified by macca8)

If the autosave slot for each installed game is either empty or contains a valid autosave saved game, then surely this warning shouldn't be displayed to the user when enabling the feature.

A picture speaks a thousand words (see attached).

Refers to commit 873ba58 - Warn when enabling autosave on options if non-autosave games are stored in autosave slot.
The commit message and code clearly target listing only non-autosave games, however, it seems that no consideration has been made for that list to be empty, thus prompting an unnecessary warning.

If I'm understanding this commit correctly, it also attempts to minimize the number of game engines that need to be tested, by constructing a sample size of 10 non-autosave games, rather than a complete list for all engines (but I may be mistaken). It follows that, if this list is empty (or less than 10), then testing will continue until all game engines have been tested, which may not be the desired outcome for those with a large number of games installed.

Attachments (1)

slots contain only valid autosaves.png (84.1 KB ) - added by macca8 5 months ago.

Download all attachments as: .zip

Change History (5)

comment:1 by macca8, 5 months ago

Description: modified (diff)

comment:2 by macca8, 5 months ago

Description: modified (diff)

comment:3 by criezy, 5 months ago

Fixed in 18ee050adf
The dialog is now only shown if normal saves are found in autosave slots.

For now I am parking the potential performance issue when a high number of games have been added to ScummVM with no or only a few normal saves in autosaves slots. If this proves to be a performance issue, then a separate ticket should be created for it.

comment:4 by macca8, 5 months ago

Owner: set to macca8
Resolution: fixed
Status: newclosed

Thank you. Issue resolved. Closing.

Note: See TracTickets for help on using tickets.