Follow-up Comment #31, patch #3829 (project freeciv):

Looking through what this touched, there's lots of "if(" in trunk: would a
cleanup patch be useful?  Only fixed the one case where one was being added,
rather than anywhere else.

I put the integrators cache initialisation into a common function.  This
requires an extra iteration over roads on ruleset read, so is a bit slower
(not that ruleset read is a meaningful optimisation target), and is more lines
of code overall.  That said, it should now be easier to make changes to this
function without needing to coordinate the behaviour in two places.

(file #20422)
    _______________________________________________________

Additional Item Attachment:

File name: allow-compatible-roads+common-cache.patch Size:18 KB


    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  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