Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-978138 into lp:widelands

2015-05-02 Thread GunChleoc
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

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1428396 into lp:widelands

2015-05-02 Thread GunChleoc
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

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-998544 into lp:widelands

2015-05-02 Thread GunChleoc
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