widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~shevonar/widelands/feature-random-tribe-and-AI into lp:widelands

To: mp+75897@xxxxxxxxxxxxxxxxxx
From: Victor Pelt <victor@xxxxxxx>
Date: Sun, 18 Sep 2011 15:46:23 -0000
In-reply-to: <20110918135322.2038.17937.launchpad@ackee.canonical.com>
Reply-to: mp+75897@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

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

[Merge] lp:~shevonar/widelands/feature-random-tribe-and-AI into lp:widelands
From: Shevonar, 2011-09-18