[Freeciv-Dev] Re: [PATCH] tile_is_known()

[Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>]

On Fri, Sep 14, 2001 at 09:49:38AM -0500, Mike Kaufman wrote:
> here is a patch to move tile_is_known() out of map.c and into the
> client.
> 
> The direct benefit is to reduce the civworld code size by 60k.
> please apply to CVS.
> 
> It is based on the Sept 12 cvs-snapshot.
> 
> -mike

> diff -Nur -Xfreecivc-tileknown/diff_ignore rawcivc/client/Makefile.am 
> freecivc-tileknown/client/Makefile.am
> --- rawcivc/client/Makefile.am        Mon Sep  3 00:55:06 2001
> +++ freecivc-tileknown/client/Makefile.am     Thu Sep 13 22:33:15 2001
> @@ -112,6 +112,8 @@
>       civclient.h     \
>       climisc.c       \
>       climisc.h       \
> +     climap.c        \
> +     climap.h        \
>       clinet.c        \
>       clinet.h        \
>       control.c       \
> diff -Nur -Xfreecivc-tileknown/diff_ignore rawcivc/client/climap.c 
> freecivc-tileknown/client/climap.c
> --- rawcivc/client/climap.c   Wed Dec 31 18:00:00 1969
> +++ freecivc-tileknown/client/climap.c        Thu Sep 13 22:46:34 2001
> @@ -0,0 +1,36 @@
> +/**********************************************************************
> + Freeciv - Copyright (C) 1996 - A Kjeldberg, L Gregersen, P Unold
> +   This program is free software; you can redistribute it and/or modify
> +   it under the terms of the GNU General Public License as published by
> +   the Free Software Foundation; either version 2, or (at your option)
> +   any later version.
> +
> +   This program is distributed in the hope that it will be useful,
> +   but WITHOUT ANY WARRANTY; without even the implied warranty of
> +   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> +   GNU General Public License for more details.
> +***********************************************************************/
> +
> +/**********************************************************************
> + this map contains client specific map functions.

s/this map/this file/?!

> + don't even think about consolidating into tilespec.c and

Why not?

Can we put this into climisc.[ch]?

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "Your mail could not be delivered to the following Address:
  VTCMC.VTLPR@xxxxxxxxxxxxx        ** Unassigned error message **"