Complete.Org: Mailing Lists: Archives: freeciv-dev: September 2004:
[Freeciv-Dev] (PR#2521) general effects framework
Home

[Freeciv-Dev] (PR#2521) general effects framework

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: kaufman@xxxxxxxxxxxxxxxxxxxxxx, vasc@xxxxxxxxxxxxxx
Subject: [Freeciv-Dev] (PR#2521) general effects framework
From: "Jason Short via RT" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Sat, 4 Sep 2004 22:08:24 -0700
Reply-to: RT_CorrespondAddressNotSet@xxxxxxxxxxxxxx

<URL: http://RT::WebBaseURL.not.configured:80/Ticket/Display.html?id=2521 >

Here is an update and new version of the effects patch.

In addition to the update, I made a lot of changes.

- Callers of get_xxx_bonus shouldn't treat it as a boolean.
- Renamed some eff variables and globals as effect.  Similarly eft gets
renamed as effect_type.  And in some places eff becomes effect_type...
- Removed an unused field from a packet.
- Merged get_xxx_bonus into one function (in the backend only).  Pretty
nifty.  I'd say this is the core of Vasco's design, and it should show
how it can be extensible.
- Added a new function ruleset_cache_free.  It is called from the server
only (the client just leaks data for now).
- Added some more explanatory comments.
- Updated and included the other rulesets.

jason

Attachment: effects16.diff.gz
Description: Unix tar archive


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