← Back to team overview

widelands-dev team mailing list archive

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