[Freeciv-Dev] Re: (PR#12833) consider_settler_action simplification

["Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>]

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

Brian Dunstan wrote:

> I think that patch is a good idea, but do not think it
> will alter the behavior much.  diff=(new_tile_value -
> old_tile_value) is also multiplied by 64 before
> amortization, so that -1 addition will be overwhelmed
> almost all of the time.  There's no chance that
> amortize() will be fed a negative number, or that a
> negative number will be returned as the discounted
> value of the improvement.

The difference is for unworked (!inuse) tiles, which I think will
(almost) all end up with a value of 0.

-jason