Complete.Org: Mailing Lists: Archives: freeciv-dev: December 2001:
[Freeciv-Dev] Re: advdomestic.c cleanup II. (PR#1157)
Home

[Freeciv-Dev] Re: advdomestic.c cleanup II. (PR#1157)

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Mike Kaufman <mkaufman@xxxxxxxxxxxxxx>
Cc: Petr Baudis <pasky@xxxxxxxxxxx>, freeciv-dev@xxxxxxxxxxx, bugs@xxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: advdomestic.c cleanup II. (PR#1157)
From: "Ross W. Wetmore" <rwetmore@xxxxxxxxxxxx>
Date: Fri, 28 Dec 2001 17:42:08 -0500

I'll pick it up and get a first review back sometime over the weekend, now
that I'm out of the topology maze :-).

Cheers,
RossW
=====

At 03:02 PM 01/12/28 -0600, Mike Kaufman wrote:
>On Mon, Dec 24, 2001 at 11:30:28AM +0100, Petr Baudis wrote:
>> Dear diary, on Mon, Dec 24, 2001 at 10:46:36AM CET, I got a letter,
>> where Raahul Kumar <raahul_da_man@xxxxxxxxx> told me, that...
>> > int want = pcity->ai.building_want[acity->currently_building];
>> > +      /* Distance to wonder city was established after manage_bu and
before
>> > +       * this.  If we just started building a wonder during a_c_c_b, the
>> > +       * started_building notify comes equipped with an update.  It
calls
>> > +       * generate_warmap, but this is a lot less warmap generation
than there
>> > +       * would be otherwise. -- Syela */
>> > +      int dist;
>> > 
>> > I have no idea what this line 
>> > 
>> > If we just started building a wonder during a_c_c_b, the started_building
>> > notify comes equipped with an update. 
>> > 
>> > a_c_c_b baffles me. If you think this actually makes else, keep it.
>> Ok, I expanded the function names here :-).
>
>Could I get a couple of reviewers (and reviews) for this patch? Ross,
>Raahul?, anyone else?
>
>-mike
>
>



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