widelands-dev team mailing list archive

[GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>]

Are you using the []? If not, the bonus doesn't gain you anything ;)

What I'm looking at is the performance loss when deleting elements from the queue, because you're not using it as a queue - queues are there so you can push and pop, so deleting is much more efficient with lists. Please have a look if you really need to do all that iteration over the container just to delete an element - the iteration might actually lose you what you are trying to gain here.
-- 
https://code.launchpad.net/~widelands-dev/widelands/ai_variables_rework/+merge/333041
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_variables_rework into lp:widelands.