widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #12866
Re: [Merge] lp:~widelands-dev/widelands/remove_deprecated_allow_autosaving into lp:widelands
Review: Approve
Disclaimer: I have no idea about the Lua bindings but no-one seems to want to review this.
Apart from that, grep couldn't find references to this function in the data directory and bunnybot says it compiles. Diff looks fine, too. Haven't bothered testing.
So the only downside to this branch that I can see is that there might be scenarios out there that breaks after the merge. But well, the function has been deprecated long enough.
--
https://code.launchpad.net/~widelands-dev/widelands/remove_deprecated_allow_autosaving/+merge/339492
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/remove_deprecated_allow_autosaving.
References