widelands-dev team mailing list archive

[TiborB <tiborb95@xxxxxxxxx>]

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 <fios@xxxxxxxxxxxxxxxxxxx>:

> 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@xxxxxxxxxxxxxxxxxxx
> 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.