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:

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

_______________________________________________
  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