Follow-up Comment #4, patch #4885 (project freeciv):
New strategy, informed by IRC discussion:
1) AI callers of is_effect_prevented() and is_improvement_redundant() to use
new daieffects facility to determine if an improvement provides any currently
useful feature (units, extras, disaster protection, action enablement, action
prevention, etc.), or could do so in the future given the capabilities of the
AI.
2) is_improvement_redundant() to be replaced with function to determine if an
improvment is currently useful (based on checking if the improvment affects
units, extras, disasters, actions, effects, etc.), which will be used to
provide UI guidance in the client, and client strings related to "redundant"
will be adjusted to better fit the new semantics.
This ticket now only strives to achieve the first of these: client-facing
changes will be in patch #4987.
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?4885>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
