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

[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: rf13@xxxxxxxxxxxxxxxxxxxxxx, 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: Arien Malec <arien_malec@xxxxxxxxx>
Date: Wed, 29 Aug 2001 07:51:21 -0700 (PDT)

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...


Do You Yahoo!?
Make international calls for as low as $.04/minute with Yahoo! Messenger

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