MJP opened the ticket and wrote:

I believe _do_autosave is set to false 1 line too early.

1/ Definition:
bool _do_autosave; ///< are we doing an autosave at the moment?

2/ In openttd.cpp, GameLoop():
if (_do_autosave) {
_do_autosave = false;
DoAutosave();

3/ DoAutosave() calls SaveOrLoad()

4/ In saveload.cpp, SaveOrLoad() tests a value always false:
/* if not an autosave, but a user action, show error message */
if (!_do_autosave) ShowErrorMessage(STR_ERROR_SAVE_STILL_IN_PROGRESS, INVALID_STRING_ID, WL_ERROR);

Attachments

• misplaced-do_autosave.patch (0.54 KiB)

Reported version: Version?
Operating system: All

This issue was imported from FlySpray: https://bugs.openttd.org/task/5871