Yes. Looks reasonable now. Needs to be tested though. From which branch did you split this one? If not from Mines_worldsavior we would probably run into a merge conflict, as we did some changes in productionsite.cc in this branch as well. So I will not approve this now although I'm happy with the result. Thanks for doing this. -- https://code.launchpad.net/~widelands-dev/widelands/bug-1786613-10s-return-skipped/+merge/353514 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/mines-worldsavior.
_______________________________________________ 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