Complete.Org: Mailing Lists: Archives: freeciv-dev: December 2001:
[Freeciv-Dev] Re: (PR#1098)
Home

[Freeciv-Dev] Re: (PR#1098)

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Vasco Alexandre Da Silva Costa <vasc@xxxxxxxxxxxxxx>
Cc: freeciv-dev@xxxxxxxxxxx, bugs@xxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#1098)
From: Mike Kaufman <mkaufman@xxxxxxxxxxxxxx>
Date: Fri, 7 Dec 2001 00:07:36 -0600

On Thu, Dec 06, 2001 at 09:02:31PM -0800, Vasco Alexandre Da Silva Costa wrote:
> Seems good. However tile_is_known() returns an enum. The name sounds like
> a predicate name.
> So wouldn't returning (map_get_tile(x,y)->known>=TILE_KNOWN_FOGGED) make
> more sense? 

no. this function needs to return the value of map_get_tile(x,y)->known
See tilespec.c for example. if this is confusing I'll resubmit the patch
changing all the instances of tile_is_known() to tile_known() or a more
suitable name if you can come up with one.

-mike

> 
> ---
> Vasco Alexandre da Silva Costa @ Instituto Superior Tecnico, Lisboa
> 


[Prev in Thread] Current Thread [Next in Thread]
  • [Freeciv-Dev] (PR#1098), Vasco Alexandre Da Silva Costa, 2001/12/06
    • [Freeciv-Dev] Re: (PR#1098), Mike Kaufman <=