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

Reply via email to