Complete.Org: Mailing Lists: Archives: freeciv-dev: October 2001:
[Freeciv-Dev] Re: semi-quick tile_is_known patch [new patch]
Home

[Freeciv-Dev] Re: semi-quick tile_is_known patch [new patch]

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: jdorje@xxxxxxxxxxxxxxxxxxxxx
Cc: freeciv-dev <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] Re: semi-quick tile_is_known patch [new patch]
From: Mike Kaufman <mkaufman@xxxxxxxxxxxxxx>
Date: Sat, 27 Oct 2001 17:09:51 -0500

> Yes.  Using map_get_tile is better.  But please change this code:
> to
> 
> {
>   return map_get_tile(x, y)->known;
> }

ok, I agree. Here's a new patch.

> This will make things simpler for the check_map_pos changes.  Speaking
> of which, since civworld uses this same code you should verify that
> civworld works with the check_map_pos patch (i.e. doesn't pass around
> non-normal coordinates anywhere).

it uses a completely stock map.c and maphand.c and a _mostly_ stock
mapview.c, so I don't foresee real problems.

> > > > This patch is needed to remove ~60k from the civworld patch.
> > >
> > > It seems like a better solution would be to put civworld into freeciv's
> > > CVS, so that source sharing would be much easier.  But I'm sure this has
> > > been discussed before...
> > 
> > this won't change anything, as civworld still uses both server and
> > client specific contructs. see above.
> 
> Yes, I see.  I also see you're in favor of it :-)
> 
> > > Finally, I'd rather see the check_map_pos patch go in before this one.
> > > It makes this fix simpler...

But, this now becomes an easy thing for Raimar to apply. Raimar: please apply
it. 

-mike

Attachment: tile_is_known_patch
Description: Text document


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