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

> is_square_threatened()

It would be nice to have these two similar functions sharing the code. I bet
that if we ever add special units that are not UTYF_DIPLOMAT (which in fact is
not telling so much any more as most actions are controlled by action
enablers), we remember to update only one of these. Could we have
is_unit_offensive() with some generic implementation (maybe the cases differ
too much for this)?


    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  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