Use unordered_map, no need for an ordered map

Author: Causeless

Managers/SettingsMan.h

@@ -375,7 +375,7 @@ namespace RTE {
 		/// Gets the map of mods which are disabled.
 		/// </summary>
 		/// <returns>Map of mods which are disabled.</returns>
-		std::map<std::string, bool> & GetDisabledModsMap() { return m_DisabledMods; }
+		std::unordered_map<std::string, bool> & GetDisabledModsMap() { return m_DisabledMods; }
 
 		/// <summary>
 		/// Gets whether the specified mod is disabled in the settings.
@@ -388,7 +388,7 @@ namespace RTE {
 		/// Gets the map of global scripts which are enabled.
 		/// </summary>
 		/// <returns>Map of global scripts which are enabled.</returns>
-		std::map<std::string, bool> & GetEnabledGlobalScriptMap() { return m_EnabledGlobalScripts; }
+		std::unordered_map<std::string, bool> & GetEnabledGlobalScriptMap() { return m_EnabledGlobalScripts; }
 
 		/// <summary>
 		/// Gets whether the specified global script is enabled in the settings.
@@ -563,8 +563,8 @@ namespace RTE {
 		bool m_MeasureModuleLoadTime; //!< Whether to measure the duration of data module loading (extraction included). For benchmarking purposes.
 
 		std::list<std::string> m_VisibleAssemblyGroupsList; //!< List of assemblies groups always shown in editors.
-		std::map<std::string, bool> m_DisabledMods; //!< Map of the module names we disabled.
-		std::map<std::string, bool> m_EnabledGlobalScripts; //!< Map of the global script names we enabled.
+		std::unordered_map<std::string, bool> m_DisabledMods; //!< Map of the module names we disabled.
+		std::unordered_map<std::string, bool> m_EnabledGlobalScripts; //!< Map of the global script names we enabled.
 
 	private:
 

Menus/ModManagerGUI.cpp

@@ -118,7 +118,7 @@ namespace RTE {
 	void ModManagerGUI::ToggleMod() {
 		int index = m_ModsListBox->GetSelectedIndex();
 		if (index > -1) {
-			std::map<std::string, bool> &disabledModsList = g_SettingsMan.GetDisabledModsMap();
+			std::unordered_map<std::string, bool> &disabledModsList = g_SettingsMan.GetDisabledModsMap();
 			GUIListPanel::Item *selectedItem = m_ModsListBox->GetSelected();
 			ModRecord &modRecord = m_KnownMods.at(selectedItem->m_ExtraIndex);
 
@@ -146,7 +146,7 @@ namespace RTE {
 	void ModManagerGUI::ToggleScript() {
 		int index = m_ScriptsListBox->GetSelectedIndex();
 		if (index > -1) {
-			std::map<std::string, bool> &enabledScriptList = g_SettingsMan.GetEnabledGlobalScriptMap();
+			std::unordered_map<std::string, bool> &enabledScriptList = g_SettingsMan.GetEnabledGlobalScriptMap();
 			GUIListPanel::Item *selectedItem = m_ScriptsListBox->GetSelected();
 			ScriptRecord &scriptRecord = m_KnownScripts.at(selectedItem->m_ExtraIndex);