Simplify module handle usage

Author: Xottab-DUTY

src/Layers/xrRender/HW.cpp

@@ -84,7 +84,7 @@ void CHW::CreateD3D()
 {
     const pcstr _name = GEnv.isDedicatedServer ? "xrD3D9-Null" : "d3d9.dll";
 
-    hD3D = std::make_unique<XRay::Module>(_name);
+    hD3D = XRay::LoadModule(_name);
 
     R_ASSERT2(hD3D->exist(), "Can't find 'd3d9.dll'\nPlease install latest version of DirectX before running this program");
     typedef IDirect3D9* WINAPI _Direct3DCreate9(UINT SDKVersion);

src/Layers/xrRender/HW.h

@@ -89,7 +89,7 @@ class CHW
     D3D_FEATURE_LEVEL FeatureLevel;
 #else
 private:
-    std::unique_ptr<XRay::Module> hD3D;
+    XRay::Module hD3D;
 
 public:
     IDirect3D9* pD3D; // D3D

src/Layers/xrRenderPC_R4/r2_test_hw.cpp

@@ -13,7 +13,7 @@ typedef HRESULT(__stdcall* FuncPtrD3D11CreateDeviceAndSwapChain)(IDXGIAdapter* p
 
 bool TestDX11Present()
 {
-    const auto hD3D11 = std::make_unique<XRay::Module>("d3d11.dll");
+    const auto hD3D11 = XRay::LoadModule("d3d11.dll");
 
     if (!hD3D11->exist())
     {

src/utils/xrAI/xrAI.cpp

@@ -102,7 +102,7 @@ void execute(LPSTR cmd)
             char* no_separator_check = strstr(cmd, "-no_separator_check");
             clear_temp_folder();
 
-            const auto hFactory = std::make_unique<XRay::Module>("xrSE_Factory");
+            const auto hFactory = XRay::LoadModule("xrSE_Factory");
 
             R_ASSERT2(hFactory->exist(), "Factory DLL raised exception during loading or there is no factory DLL at all");
 

src/utils/xrLC/xrLC.cpp

@@ -93,7 +93,7 @@ void Startup(LPSTR lpCmdLine)
     if (bModifyOptions)
     {
         Logger.Phase("Project options...");
-        const auto L = std::make_unique<XRay::Module>("xrLC_Options");
+        const auto L = XRay::LoadModule("xrLC_Options");
 
         const auto O = (xrOptions*)L->getProcAddress("_frmScenePropertiesRun");
         R_ASSERT(O);

src/utils/xrSE_Factory/properties_list_helper_script.cpp

@@ -21,16 +21,16 @@ struct CChooseType
 {
 };
 
-typedef IPropHelper&(__stdcall* TPHelper)();
+using TPHelper = IPropHelper&(__stdcall* )();
 
 TPHelper _PHelper = nullptr;
-std::unique_ptr<XRay::Module> prop_helper_module;
+XRay::Module prop_helper_module;
 constexpr pcstr prop_helper_library = "xrEPropsB", prop_helper_func = "PHelper";
 CScriptPropertiesListHelper* g_property_list_helper = nullptr;
 
 void load_prop_helper()
 {
-    prop_helper_module = std::make_unique<XRay::Module>(prop_helper_library);
+    prop_helper_module = XRay::LoadModule(prop_helper_library);
     if (!prop_helper_module->exist())
     {
         Msg("! Cannot find library %s", prop_helper_library);

src/utils/xrSE_Factory/xrSE_Factory.cpp

@@ -20,7 +20,7 @@
 extern CSE_Abstract* F_entity_Create(LPCSTR section);
 
 extern CScriptPropertiesListHelper* g_property_list_helper;
-extern std::unique_ptr<XRay::Module> prop_helper_module;
+extern XRay::Module prop_helper_module;
 
 extern "C" {
 FACTORY_API IServerEntity* __stdcall create_entity(LPCSTR section) { return F_entity_Create(section); }

src/xrCore/ModuleLookup.cpp

@@ -4,19 +4,19 @@
 
 namespace XRay
 {
-Module::Module(const bool dontUnload) : handle(nullptr), dontUnload(dontUnload) {}
+ModuleHandle::ModuleHandle(const bool dontUnload) : handle(nullptr), dontUnload(dontUnload) {}
 
-Module::Module(pcstr moduleName, bool dontUnload /*= false*/) : handle(nullptr), dontUnload(dontUnload)
+ModuleHandle::ModuleHandle(pcstr moduleName, bool dontUnload /*= false*/) : handle(nullptr), dontUnload(dontUnload)
 {
     open(moduleName);
 }
 
-Module::~Module()
+ModuleHandle::~ModuleHandle()
 {
     close();
 }
 
-void* Module::open(pcstr moduleName)
+void* ModuleHandle::open(pcstr moduleName)
 {
     if (exist())
         close();
@@ -31,7 +31,7 @@ void* Module::open(pcstr moduleName)
     return handle;
 }
 
-void Module::close()
+void ModuleHandle::close()
 {
     if (dontUnload)
         return;
@@ -40,18 +40,18 @@ void Module::close()
     handle = nullptr;
 }
 
-bool Module::exist() const
+bool ModuleHandle::exist() const
 {
     return handle != nullptr;
 }
 
-void* Module::operator()() const
+void* ModuleHandle::operator()() const
 {
     return handle;
 }
 
-void* Module::getProcAddress(pcstr procName) const
+void* ModuleHandle::getProcAddress(pcstr procName) const
 {
-    return ::GetProcAddress(static_cast<HMODULE>(handle), procName);
+    return GetProcAddress(static_cast<HMODULE>(handle), procName);
 }
 }

src/xrCore/ModuleLookup.hpp

@@ -1,16 +1,17 @@
 #pragma once
+#include <memory>
 
 namespace XRay
 {
-class XRCORE_API Module
+class XRCORE_API ModuleHandle
 {
     void* handle;
     bool dontUnload;
 
 public:
-    Module(const bool dontUnload = false);
-    Module(pcstr moduleName, bool dontUnload = false);
-    ~Module();
+    ModuleHandle(const bool dontUnload = false);
+    ModuleHandle(pcstr moduleName, bool dontUnload = false);
+    ~ModuleHandle();
 
     void* open(pcstr moduleName);
     void close();
@@ -21,4 +22,16 @@ class XRCORE_API Module
 
     void* getProcAddress(pcstr procName) const;
 };
+
+using Module = std::unique_ptr<ModuleHandle>;
+
+inline auto LoadModule(bool dontUnload = false)
+{
+    return std::make_unique<ModuleHandle>(dontUnload);
+}
+
+inline auto LoadModule(pcstr moduleName, bool dontUnload = false)
+{
+    return std::make_unique<ModuleHandle>(moduleName, dontUnload);
+}
 }

src/xrEngine/Device_Initialize.cpp

@@ -10,7 +10,7 @@ extern LRESULT CALLBACK WndProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lPar
 
 void CRenderDevice::initialize_editor()
 {
-    m_editor_module = std::make_unique<XRay::Module>("xrWeatherEditor");
+    m_editor_module = XRay::LoadModule("xrWeatherEditor");
     if (!m_editor_module->exist())
         return;