Complete.Org: Mailing Lists: Archives: freeciv-ai: March 2005:
[freeciv-ai] Re: (PR#12422) stupid AI leaves its city in disorder

[freeciv-ai] Re: (PR#12422) stupid AI leaves its city in disorder

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: jdorje@xxxxxxxxxxxxxxxxxxxxx
Subject: [freeciv-ai] Re: (PR#12422) stupid AI leaves its city in disorder
From: "Benoit Hudson" <bh@xxxxxxxxxxxxxxxxxxx>
Date: Fri, 4 Mar 2005 18:48:04 -0800
Reply-to: bugs@xxxxxxxxxxx

<URL: >

On Fri, Mar 04, 2005 at 05:39:35PM -0800, Jason Short wrote:
> <URL: >
> > [per - Thu Mar 03 23:50:47 2005]:
> > 
> > On Thu, 3 Mar 2005, Jason Short wrote:
> > > Not sure about the bug but your assert is wrong because there is a
> > > generic_city_refresh call built in at the start of CM.Here is a more
> > > comprehensive and correct assert.
> > 
> > No, the assert is correct. The generic_city_refresh call calculates the
> > city state wrong. The city should be in disorder - it is not.
> The assert is surely not correct.  Because the callers pass in cities
> that aren't refreshed the refresh _will_ sometimes change this value.  I
> also find it unlikely that generic_city_refresh is wrong here - this
> function is called so often if it were wrong the whole game would explode.
> However there is certainly a bug in ai_manage_taxes.  I think this patch
> fixes it (the comment explains the bug).  I think it's in S2_0 as well.

cm_query_result should never be clobbering the city; I had noted that
your patch was buggy in #11415 and offered a patch but probably you
didn't see it since apparently the bug was already marked closed.

        -- Benoît

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