Follow-up Comment #2, patch #4990 (project freeciv):

Testing this more with for patch #4987 shows that
improvementhas_side_effects() almost never returns TRUE with conjunctive
logic, and that the function should really check if ANY side effect is
available, rather than if ALL side effects are available.  Fix attached.

(file #21570)
    _______________________________________________________

Additional Item Attachment:

File name:
consider-improvement-side-effects-other-than-buildable-units+disjunction.patch
Size:13 KB


    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  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