Follow-up Comment #2, patch #4821 (project freeciv):
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:
Message sent via/by Gna!
Freeciv-dev mailing list