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.
Additional Item Attachment:
File name: allow-compatible-roads+common-cache.patch Size:18 KB
Reply to this item at:
Message sent via/by Gna!
Freeciv-dev mailing list