[Freeciv-Dev] Re: (PR#2575) define CITY_TILES in city.h

["Raimar Falke via RT" <rt@xxxxxxxxxxxxxx>]

On Wed, Dec 18, 2002 at 01:30:44PM +0000, Gregory Berkolaiko wrote:
> I located another use for this define.
> 
> In client/agents/cma_core.c there is 
> #define MAX_FIELDS_USED         (CITY_MAP_SIZE * CITY_MAP_SIZE - 4 - 1)

I know ;)

> which is actually only used as (MAX_FIELDS_USED + 1).
> 
> There are following options: 
> 1. MAX_FIELDS_USED + 1 is substituted with CITY_TILES throughtout 
> cma_core.c and the define is removed
> 2. MAX_FIELDS_USED is left where it is, but the define is changed to 
> #define MAX_FIELDS_USED         (CITY_TILES - 1)

> Raimar, which you would prefer.

2.

> In general, when I know Raimar's opinion, I intend to commit the
> patch straight away.  So speak up or forever be silent.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
  Living on earth may be expensive, but it includes an annual free trip
  around the sun.