Complete.Org: Mailing Lists: Archives: freeciv-dev: April 2005:
[Freeciv-Dev] Re: (PR#10702) RFC: removing ptile->move_cost
Home

[Freeciv-Dev] Re: (PR#10702) RFC: removing ptile->move_cost

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: jdorje@xxxxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#10702) RFC: removing ptile->move_cost
From: "Benoit Hudson" <bh@xxxxxxxxxxxxxxxxxxx>
Date: Tue, 12 Apr 2005 07:58:01 -0700
Reply-to: bugs@xxxxxxxxxxx

<URL: http://bugs.freeciv.org/Ticket/Display.html?id=10702 >

On Tue, Apr 12, 2005 at 12:06:58AM -0700, Jason Short wrote:
> 
> <URL: http://bugs.freeciv.org/Ticket/Display.html?id=10702 >
> 
> Here's an updated patch to remove ptile->move_costs.  This one is
> intended to be committed.
> 
> Again, the reason for removing the move_costs is that it's a fairly
> bug-prone cache that doesn't actually speed things up very much. 
> Various comments throughout the code (which are removed by the patch)
> should make this clear.

tile_move_cost_ai has nothing to do with the AI, so just call it
tile_move_cost.  In fact, it's not local to a single tile, so it should
probably be map_move_cost.

        -- Benoît





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