Changes looks good unless I missed something realy bad :) Part of changes was modification of proportion of in construction vs finished militarysites. Now the rules are bit relaxed. So if there was enemy nearby and total number of military constructions sites were not that high it might be OK, I will run the test on the same map to see if it behaves as intended.
2015-05-07 14:40 GMT+02:00 GunChleoc <f...@foramnagaidhlig.net>: > I added some small code style tweaks - please have a look to see if you > like the changes. > > While testing with Twin Lagoons, the blue player built 3 Donjons and a > Fortress right next to each other to the south of the mountain. This didn't > happen before - can you have another look at your code to find it? The math > is a bit over my head ;) > > -- > > https://code.launchpad.net/~widelands-dev/widelands/militarysites_tweaks/+merge/255132 > Your team Widelands Developers is requested to review the proposed merge > of lp:~widelands-dev/widelands/militarysites_tweaks into lp:widelands. > > _______________________________________________ > Mailing list: https://launchpad.net/~widelands-dev > Post to : widelands-dev@lists.launchpad.net > Unsubscribe : https://launchpad.net/~widelands-dev > More help : https://help.launchpad.net/ListHelp > -- https://code.launchpad.net/~widelands-dev/widelands/militarysites_tweaks/+merge/255132 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/militarysites_tweaks into lp:widelands. _______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
