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
[email protected]
https://mail.gna.org/listinfo/freeciv-dev
