Complete.Org: Mailing Lists: Archives: freeciv-dev: October 2004:
[Freeciv-Dev] Re: (PR#10368) RfP: #define ACTIVITY_FACTOR
Home

[Freeciv-Dev] Re: (PR#10368) RfP: #define ACTIVITY_FACTOR

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: jdorje@xxxxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#10368) RfP: #define ACTIVITY_FACTOR
From: "Per I. Mathisen" <per@xxxxxxxxxxx>
Date: Sun, 17 Oct 2004 04:05:39 -0700
Reply-to: rt@xxxxxxxxxxx

<URL: http://rt.freeciv.org/Ticket/Display.html?id=10368 >

On Sat, 16 Oct 2004, Jason Short wrote:
> The *10 factor for activities was added in:
>
> date: 2004/01/11 17:45:04;author: per;  state: Exp;  lines: +9 -9
> Add support for multiple veteran levels, defined in units.ruleset. See
> new documentation in units.ruleset for more info. Patch by James Blewitt
> <james.blewitt@xxxxxxxxxxxxxx> Rafal Bursig <bursig@xxxxxxxxx> and me.
>
> However there is no documentation of why it would be needed and no PR#
> to look up the original discussion.So I have no idea why it is needed.
>  Per, can you shed any light on this?

IIRC, it was not added in this patch, merely extended to cover new cases.
There was already some places where it was used, and the choice was to
either rewrite this code, or make the new code to match. But I could be
remembering wrongly.

  - Per




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