URL:
  <http://gna.org/bugs/?21419>

                 Summary: Conjunctive requirement sanity checking applied to
disjunctive building obsolete_by list
                 Project: Freeciv
            Submitted by: jtn
            Submitted on: Fri Jan  3 01:45:14 2014
                Category: None
                Severity: 3 - Normal
                Priority: 5 - Normal
                  Status: In Progress
             Assigned to: jtn
        Originator Email: 
             Open/Closed: Open
                 Release: 
         Discussion Lock: Any
        Operating System: Any
         Planned Release: 

    _______________________________________________________

Details:

In sanity_check_ruleset_data(), sanity_check_req_vec() is used for all
requirements lists, including pimprove->obsolete_by.
However, obsolete_by is a disjunctive ("or") list -- any of the requirements
is sufficient to obsolete the building. The checks currently in
sanity_check_req_vec() are only appropriate for "and" lists.
For instance, it's a bit of a stretch, but it would be possible to allow
either of two different Govs to obsolete a building, but the sanity checking
forbids it. (Two techs are more likely, but the ruleset checking allows
this.)

(Not that complex obsolete reqs are currently handled particularly gracefully
elsewhere, particularly in the help, but it would work in gameplay terms.)




    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?21419>

_______________________________________________
  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