xrEngine/EngineAPI.cpp: no longer need to copy renderer mode strings

Libraries aren't unloaded anymore after
f351ad4

Author: Xottab-DUTY

src/xrEngine/EngineAPI.cpp

@@ -9,7 +9,6 @@
 #include "XR_IOConsole.h"
 
 #include "xrCore/xr_token.h"
-#include "xrCore/ModuleLookup.hpp"
 
 #include "xrScriptEngine/ScriptExporter.hpp"
 
@@ -72,7 +71,7 @@ void CEngineAPI::SelectRenderer()
             if (renderer->CheckGameRequirements())
             {
                 selectedRenderer = renderer;
-                selected_mode = mode.c_str();
+                selected_mode = mode;
                 string64 buf;
                 xr_sprintf(buf, "renderer %s", selected_mode);
                 Console->Execute(buf);
@@ -143,10 +142,8 @@ void CEngineAPI::CreateRendererList(const std::array<RendererModule*, 2>& module
                 VERIFY3(false, "Renderer mode duplicate. Skipping.", mode);
                 continue;
             }
-            // mode string will be freed after library unloading, copy.
-            shared_str copiedMode = mode;
             renderModes[mode] = module;
-            VidQualityToken.emplace_back(copiedMode.c_str(), modeIndex);
+            VidQualityToken.emplace_back(mode, modeIndex);
         }
 
         return true;

src/xrEngine/EngineAPI.h

@@ -62,7 +62,7 @@ class XR_NOVTABLE RendererModule
 
 class ENGINE_API CEngineAPI
 {
-    xr_map<shared_str, RendererModule*> renderModes;
+    xr_map<cpcstr, RendererModule*> renderModes;
 
     GameModule* gameModule{};
     RendererModule* selectedRenderer{};