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 Nov 28, 2023
1 parent ec926a6 commit 0118e05
Show file tree
Hide file tree
Showing 3 changed files with 30 additions and 1 deletion.
8 changes: 8 additions & 0 deletions metamod/include/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 metamod/include/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
21 changes: 20 additions & 1 deletion metamod/src/meta_rehlds_api.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,26 @@ bool rehlds_api_init(CSysModule* engineModule)

bool meta_init_rehlds_api()
{
CSysModule* engineModule = Sys_LoadModule(ENGINE_LIB);
#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", "hw.dll", "sw.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)
{
if (engineModule = Sys_GetModuleHandle(dllName))
break; // gotcha
}

#else
CSysModule *engineModule = Sys_GetModuleHandle("engine_i486.so");
#endif

if (!rehlds_api_init(engineModule)) {
return false;
}
Expand Down

0 comments on commit 0118e05

Please sign in to comment.