Complete.Org: Mailing Lists: Archives: freeciv-dev: February 2002:
[Freeciv-Dev] Re: [Patch] for to players_iterate
Home

[Freeciv-Dev] Re: [Patch] for to players_iterate

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: rf13@xxxxxxxxxxxxxxxxxxxxxx
Cc: freeciv development list <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] Re: [Patch] for to players_iterate
From: Tony Stuckey <stuckey@xxxxxxxxxxxxxxxxx>
Date: Thu, 21 Feb 2002 18:21:16 -0600

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.


[Prev in Thread] Current Thread [Next in Thread]