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

                 Summary: Use UTYF_BADCITYDEFENDER in process_defender_want()
                 Project: Freeciv
            Submitted by: persia
            Submitted on: Mon 23 Jun 2014 09:43:47 AM JST
                Category: ai
                Priority: 5 - Normal
                  Status: Ready For Test
                 Privacy: Public
             Assigned to: persia
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: 2.6.0

    _______________________________________________________

Details:

When UTYF_BADCITYDEFENDER was introduced in r10826, it appears that
process_defender_want() was not updated to use the new flag, and instead used
the old hardcoding to determine whether a given unit interacted sensibly with
walls.  This hasn't been a bug, mostly because ruleset authors haven't
assigned DefendOK or DefendGood to UMT_SEA or UMT_BOTH units, but for rulesets
embracing complex nativity, it is more reasonable to restrict want based on
the flag than on the unit nativity (although it would be an oddity for a
ruleset author to give a DefendGood or DefendOK role to a unit type with a
BadCityDefender flag, so this may never be triggered in actual play).




    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Mon 23 Jun 2014 09:43:47 AM JST  Name:
use-UTYF_BADCITYDEFENDER-in-process_defender_want.patch  Size: 3kB   By:
persia

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

    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  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