widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #08724
Re: [Merge] lp:~7010622-q/widelands/topple-seafaring-1 into lp:widelands
iterating over existing portspaces makes sense, because:
- the ship does not pick first portspace it finds, but picks good one with some randomness, next time it visit the portspace it can decide to build a port there
- When all existing portspaces are know, the ship can still pick one of them to build a port, otherwise it can end up looking for new portspaces forever. If I recall properly - cancellation of expedition does not work, so the last one lasts forever
--
https://code.launchpad.net/~7010622-q/widelands/topple-seafaring-1/+merge/310436
Your team Widelands Developers is requested to review the proposed merge of lp:~7010622-q/widelands/topple-seafaring-1 into lp:widelands.
References