id	summary	reporter	owner	description	type	status	priority	milestone	component	resolution	keywords	cc	phab_field
2012	[PATCH] Cache GetPercentMapExplored	scroogie	ben	"In a callgrind run I've seen that CCmpRangeManager::GetPercentMapExplored showed up as an expensive function (callgrind showed 6.56 self). Its apparently called quite often for statistics gathering. The patch changes the implementation to cache the value instead of recomputing it every time, as that includes traversing all tiles of the map. Also the original implementation traverses all tiles for playerId==0, which is not required, as it will always be 0. This gave me a slightly measurable speedup in replays of full games. Its of course only faster if the function is actually called that often, of which I didn't know if its on purpose or not. Should it only be called at the end of a game?
Anyway, in case it stays like this, here is a patch."	enhancement	reopened	Nice to Have	Alpha 14	Core engine		patch performance	Itms wraitii