← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/bug-1685331-always-enable into lp:widelands

 

Review: Approve

Code looks good and it seems to work fine. Tested it with b19 savegames with and without ports and with a current scenario savegame.
A small remark: Increasing "tribe_buildings" could probably be replaced with "player->tribe().buildings().size()".
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1685331-always-enable/+merge/323481
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1685331-always-enable.


References