widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/robust-saving into lp:widelands

To: mp+358220@xxxxxxxxxxxxxxxxxx
From: Arty <diego.scheide@xxxxxx>
Date: Fri, 02 Nov 2018 20:17:35 -0000
In-reply-to: <154115744561.29736.4396227908503512153.launchpad@ackee.canonical.com>
Reply-to: mp+358220@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Oops, I just realized that I my commit log isn't quite correct.

GameMainMenuSaveGame actually uses GenericSaveHandler directly, not via SaveHandler. I did it this way, so the formatted message (to show the player if something went wrong) could be accessed without having to change SaveHandler to allow routing it through there. Basically SaveHandler is now only used for autosaves (timed and requested via lua scripts) and to store the current filename for use in the Save Menu.
-- 
https://code.launchpad.net/~widelands-dev/widelands/robust-saving/+merge/358220
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/robust-saving into lp:widelands.

References

[Merge] lp:~widelands-dev/widelands/robust-saving into lp:widelands
From: Arty, 2018-11-02