Review: Needs Fixing I did review the code and added a bunch of comments. I think there is one bug in there.
> Allow setting of number of files (1-10) in advanced options dialog I do not think we want users to mess with these settings and having them tunable from the console is a reasonable choice for developer options. I think we should remove the GUI part of this change - do you agree? -- https://code.launchpad.net/~widelands-dev/widelands/rolling_autosave/+merge/246061 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/rolling_autosave. _______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
