URL: <http://gna.org/patch/?4564>
Summary: Call is_airunit_refuel_point() only when needed Project: Freeciv Submitted by: cazfi Submitted on: Sat 01 Mar 2014 11:04:33 PM EET Category: ai Priority: 5 - Normal Status: None Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: _______________________________________________________ Details: In patch #4563 I said: "Profiling server running autogame shows checking if tile is refuel point as one of the functions taking most time." That is surprising also because the function should be needed for a couple of unit types only. It's probably being called too much. All the callers should first make the inexpensive checks like that unit is using fuel at all before calling expensive is_airunit_refuel_point(). _______________________________________________________ Reply to this item at: <http://gna.org/patch/?4564> _______________________________________________ Message sent via/by Gna! http://gna.org/ _______________________________________________ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev