Review: Approve
LGTM :)
--
https://code.launchpad.net/~widelands-dev/widelands/bug-978138/+merge/257005
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-978138.
___
Mailing list: https://launchpad.net/~widelands-d
You found it :)
How about only writing the ships that the player owns as well? We still need
the test when loading for older savegames, but not writing it in the first
place would be a good idea I think.
Around line 137:
if (ship->get_economy() == temp_economy) {
We could have:
if (ship->get
Fixed :)
Is this ready now?
--
https://code.launchpad.net/~widelands-dev/widelands/bug-998544/+merge/251947
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-998544.
___
Mailing list: https://launchpad.net/~widelan
3 matches
Mail list logo