[Freeciv-Dev] Re: [Patch] for to players_iterate
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
On Wed, Feb 20, 2002 at 12:05:52AM +0100, Raimar Falke wrote:
> Can somebody look at
> the change of ai_manage_diplomat? Especially what
>
> if (!ctarget && aplayer) {
>
> should mean?
Ctarget is the city that this diplomat would want to target. It is
not guaranteed to be set to a useful value. If it is unset at this point,
we haven't found a useful foreign city to do anything with.
Aplayer may or may not be guaranteed to be set, I haven't
retraced all code paths. This might be there to avoid segfaults when
dereferencing aplayer->cities immediately after this line.
What exactly are you asking?
--
Anthony J. Stuckey stuckey@xxxxxxxxxxxxxxxxx
"And they said work hard, and die suddenly, because it's fun."
-Robyn Hitchcock.
|
|