widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/ai_variables_rework into lp:widelands

To: mp+333041@xxxxxxxxxxxxxxxxxx
From: TiborB <tiborb95@xxxxxxxxx>
Date: Wed, 01 Nov 2017 20:19:31 -0000
In-reply-to: <150947174609.26585.9072313639589772525.launchpad@ackee.canonical.com>
Reply-to: mp+333041@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

The problem with list is that everytime item is added/removed the memory is allocated/deallocated. The deques and vectors behave differently. And this is the main goal - reduce number of memory de/allocations.
Also I saw some benchmarks and std::list were quite slow - depending on what you are doing with them of course.
Also, deque can access members via [] and this is nice bonus...

I added the static keyword.
-- 
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.

References

[Merge] lp:~widelands-dev/widelands/ai_variables_rework into lp:widelands
From: TiborB, 2017-10-31