Skip to content

Commit

Permalink
Improved initialization of rehlds api
Browse files Browse the repository at this point in the history
Prefer Sys_GetModuleHandle over Sys_LoadModule to avoid unnecessary reference count increase
  • Loading branch information
s1lentq committed Oct 9, 2023
1 parent ed57d7b commit 1a72c8a
Show file tree
Hide file tree
Showing 4 changed files with 25 additions and 6 deletions.
8 changes: 8 additions & 0 deletions reapi/include/cssdk/public/interface.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,14 @@ void *Sys_GetProcAddress(void *pModuleHandle, const char *pName)
return GetProcAddress((HMODULE)pModuleHandle, pName);
}

// Purpose: Returns a module handle by its name.
// Input : pModuleName - module name
// Output : the module handle or NULL in case of an error
CSysModule *Sys_GetModuleHandle(const char *pModuleName)
{
return reinterpret_cast<CSysModule *>(GetModuleHandle(pModuleName));
}

// Purpose: Loads a DLL/component from disk and returns a handle to it
// Input : *pModuleName - filename of the component
// Output : opaque handle to the module (hides system dependency)
Expand Down
2 changes: 2 additions & 0 deletions reapi/include/cssdk/public/interface.h
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,8 @@ extern CreateInterfaceFn Sys_GetFactory(const char *pModuleName);
// load/unload components
class CSysModule;

extern CSysModule *Sys_GetModuleHandle(const char *pModuleName);

// Load & Unload should be called in exactly one place for each module
// The factory for that module should be passed on to dependent components for
// proper versioning.
Expand Down
2 changes: 1 addition & 1 deletion reapi/src/mods/mod_regamedll_api.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ bool RegamedllApi_Init()
if (!szGameDLLModule)
return false;

CSysModule *gameModule = Sys_LoadModule(szGameDLLModule);
CSysModule *gameModule = Sys_GetModuleHandle(szGameDLLModule);
if (!gameModule)
return false;

Expand Down
19 changes: 14 additions & 5 deletions reapi/src/mods/mod_rehlds_api.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,15 +8,24 @@ IRehldsServerStatic* g_RehldsSvs;

bool RehldsApi_Init()
{
if (!IS_DEDICATED_SERVER())
#ifdef WIN32
// Find the most appropriate module handle from a list of DLL candidates
// Notes:
// - "swds.dll" is the library Dedicated Server
//
// Let's also attempt to locate the ReHLDS API in the client's library
// - "sw.dll" is the client library for Software render, with a built-in listenserver
// - "hw.dll" is the client library for Hardware render, with a built-in listenserver
const char *dllNames[] = { "swds.dll", "sw.dll", "hw.dll" }; // List of DLL candidates to lookup for the ReHLDS API
CSysModule *engineModule = NULL; // The module handle of the selected DLL
for (const char *dllName : dllNames)
{
return false;
if (engineModule = Sys_GetModuleHandle(dllName))
break; // gotcha
}

#ifdef WIN32
CSysModule* engineModule = Sys_LoadModule("swds.dll");
#else
CSysModule* engineModule = Sys_LoadModule("engine_i486.so");
CSysModule *engineModule = Sys_GetModuleHandle("engine_i486.so");
#endif

if (!engineModule)
Expand Down

0 comments on commit 1a72c8a

Please sign in to comment.