widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #00351
Re: [Merge] lp:~shevonar/widelands/feature-random-tribe-and-AI into lp:widelands
a few comments.
1) why did you pick an extra bool for the random tribe/ai. why not pick a tribe name called random that later gets changed to the 'real' tribe that is choosen
2) did you take in account that during a network game random will give different results for differnent people? i havn't tried your code but i know randomness can cause problems
--
https://code.launchpad.net/~shevonar/widelands/feature-random-tribe-and-AI/+merge/75897
Your team Widelands Developers is requested to review the proposed merge of lp:~shevonar/widelands/feature-random-tribe-and-AI into lp:widelands.
References