Complete.Org: Mailing Lists: Archives: freeciv-dev: July 2002:
[Freeciv-Dev] Re: RFC: Artillery Patch
Home

[Freeciv-Dev] Re: RFC: Artillery Patch

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Per I Mathisen <per@xxxxxxxxxxx>
Cc: freeciv development list <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] Re: RFC: Artillery Patch
From: Ben Webb <ben@xxxxxxxxxxxxxxxxxxxxxx>
Date: Tue, 2 Jul 2002 18:30:52 +0100

On Sat, Jun 29, 2002 at 08:57:25PM +0200, Per I Mathisen wrote:
> A unit must be able to be member of several classes at once. A horseman
> must be both "Land" and "Horse". Bonuses/penalties should be cumulative.
> 
> For example:
> 
> [unit_horsemen]
> name          = _("Horsemen")
> move_type     = "Land"
> classes       = "Land", "Horse"

        See my other post in this thread. Current impr-gen patched CVS does
almost exactly this. The only difference is that a "class" encompasses
user-defined classes, plus the existing "flags" and "move_type" fields,
so in this example you wouldn't need to explicitly list "Land" as one of
the classes.

> This should replace the following (unused gen. impr.) ugliness:

Hey, it's only unused in current CVS. impr-gen uses it, although as of
this time last week, I'm starting to phase it out.

> > > This is something that could well be handled by a "gen. units" patch.
> > Where is the developer for this patch. It's not going to be me.
> I'm hoping Ben will jump to the challenge once gen. impr. is in :)

        Actually, I thought I volunteered for this task months ago. If
not, then I do so now. ;) As Per mentioned, there is at least
rudimentary support for this at http://freecivac.sf.net/ now.

        Ben
-- 
ben@xxxxxxxxxxxxxxxxxxxxxx           http://bellatrix.pcl.ox.ac.uk/~ben/
"I think there is a world market for maybe five computers."
        - Thomas Watson, chairman of IBM, 1943


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