URL:
<http://gna.org/patch/?4938>
Summary: Consolidate server precalcs and cache population
Project: Freeciv
Submitted by: persia
Submitted on: Sun 13 Jul 2014 07:13:03 AM JST
Category: general
Priority: 5 - Normal
Status: Ready For Test
Privacy: Public
Assigned to: persia
Originator Email:
Open/Closed: Open
Discussion Lock: Any
Planned Release: 2.6.0
_______________________________________________________
Details:
In the client, most of the precalculation and cache population happens in
handle_rulesets_ready(), but in the server, it happens in several places,
making the order in which ruleset files are loaded very important (as there
are subtle bugs with caches not being properly populated for the wrong order),
and needing care to avoid use-without-assignment issues (such as bug #22014 or
problems with set_unit_class_caches() introduced with patch #4649 ).
This patch consolidates that in load_rulesetdir(), and further moves the
improvement feature cache to improvement.c, rather than being duplicate logic
in ruleset.c and packhand.c.
I'm not quite sure why role_unit_precalcs() is in set_client_state() rather
than handle_rulesets_ready(), so I've left it there for now.
_______________________________________________________
File Attachments:
-------------------------------------------------------
Date: Sun 13 Jul 2014 07:13:03 AM JST Name:
consolidate-server-precalcs-and-cache-population.patch Size: 5kB By: persia
<http://gna.org/patch/download.php?file_id=21383>
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?4938>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
