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

Things that could be wrong:
 * the cache is a static array looked up by unit id. Alternatively it could be
a filed in the unit struct or some other data structure. (At the moment only
some unit types can perform action enabler controlled actions)
 * the cache is eagerly filled on ruleset load in stead of being filled
lazily. (At the moment only one unit state property is checked)
 * it lives in unittypes.c/h and not in actions.c/h


Reply to this item at:


  Message sent via/by Gna!

Freeciv-dev mailing list

Reply via email to