As mentioned in the discussion with Vladislav today, this map can become unordered.
Test the performance, for example with the CChart.cpp diff.
Successful build - Chance fights ever on the side of the prudent.
Linter detected issues: Executing section Source... source/gui/IGUIObject.h | 44| template·<typename·T>·class·GUI; | | [MAJOR] CPPCheckBear (syntaxError): | | Code 'template<...' is invalid C code. Use --std or --language to configure the language. Executing section JS... Executing section cli...
According to Vladislavs tests, unordered_map is slower.
12:52 < Vladislav> elexis: unordered_map is slower for me, 1.2ms vs 0.9ms (map)
This patch was used by Vladislav to measure:
Well it makes sense to use an ordered map, because it allows for logarithmic instead of linear lookup time.