← Back to team overview

widelands-dev team mailing list archive

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