URL:
  <http://gna.org/patch/?3881>

                 Summary: Use an effect to control fortification capability
                 Project: Freeciv
            Submitted by: persia
            Submitted on: Thu 25 Apr 2013 03:35:57 AM JST
                Category: general
                Priority: 5 - Normal
                  Status: None
                 Privacy: Public
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: 

    _______________________________________________________

Details:

    The current code has a hardcoded is_ocean() call in the gate conditional
for ACTIVITY_FORTIFYING, meaning that UCF_CAN_FORTIFY units can fortify on any
land terrain and never fortify on any oceanic terrain.  The attached patch
changes this conditional to use an effect, supporting both more expressiveness
in ruleset definition and wider scopes for complex nativity.  Some examples of
things the patch enables:

Submarines may fortify in Deep Ocean if the player has researched the
Rebreathing technology.

Amphibious Assault Vehicles may fortify on land other than mountains or hills,
but wallow in water and may not fortify even in the presence of a river.

Big Land units cannot fortify on Farmland.

    This patch depends on the exposure of is_effect_active() from patch #3871,
but it is trivial to restack the exposure if anyone thinks this should be
applied with greater priority than that.

    Note also that the construction in helpdata.c here and the construction in
city.c there are sufficiently parallel that if anything else wants to do this
class of RPT_POSSIBLE test, it may be better to define a simple function for
that specific purpose, rather than exposing is_effect_active()



    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Thu 25 Apr 2013 03:35:57 AM JST  Name:
fortification-possible-effect.patch  Size: 20kB   By: persia

<http://gna.org/patch/download.php?file_id=17827>

    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?3881>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to