Complete.Org: Mailing Lists: Archives: freeciv-dev: August 2001:
[Freeciv-Dev] Re: [PATCH]refactor server/cityturn.c:city_build_stuff (PR
Home

[Freeciv-Dev] Re: [PATCH]refactor server/cityturn.c:city_build_stuff (PR

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Arien Malec <arien_malec@xxxxxxxxx>
Cc: freeciv-dev@xxxxxxxxxxx, bugs@xxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: [PATCH]refactor server/cityturn.c:city_build_stuff (PR#917)
From: Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>
Date: Wed, 29 Aug 2001 16:58:20 +0200
Reply-to: rf13@xxxxxxxxxxxxxxxxxxxxxx

On Wed, Aug 29, 2001 at 07:51:21AM -0700, Arien Malec wrote:
> Yes, the diff tries to be too clever, and thus does not make things very 
> clear.
> I lifted the code for city_build_stuff and put it into three functions,
> corresponding to the (hidden) flow of city_build_stuff:
> 
> distribute surplus shields
> 
> if building
>   build building
> if unit
>   build unit
> 
> If you apply the diff, all should become clear. If it helps, I can send the
> changed section, which just replaces city_build_stuff...

Objections anybody?

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "Transported to a surreal landscape, a young girl kills the first woman
  she meets and then teams up with three complete strangers to kill again."
    -- TV listing for the Wizard of Oz in the Marin Independent Journal


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