ShuriZma
/
pcsx2
mirror of https://github.com/PCSX2/pcsx2.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

DInputSource: Use wil on m_dinput_module

This commit is contained in:
Silent 2023-01-24 22:55:15 +01:00 committed by refractionpcsx2
parent c4bf297f42
commit 6a40959f3a
2 changed files with 11 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
pcsx2/Frontend/DInputSource.cpp
View File

@ -32,13 +32,7 @@ using PFNGETDFDIJOYSTICK = LPCDIDATAFORMAT(WINAPI*)();
DInputSource::DInputSource() = default;
DInputSource::~DInputSource()
{
m_controllers.clear();
m_dinput.reset();
if (m_dinput_module)
FreeLibrary(m_dinput_module);
}
DInputSource::~DInputSource() = default;
std::array<bool, DInputSource::NUM_HAT_DIRECTIONS> DInputSource::GetHatButtons(DWORD hat)
{
@ -69,15 +63,15 @@ static constexpr std::array<const char*, DInputSource::NUM_HAT_DIRECTIONS> s_hat
bool DInputSource::Initialize(SettingsInterface& si, std::unique_lock<std::mutex>& settings_lock)
{
m_dinput_module = LoadLibraryW(L"dinput8");
m_dinput_module.reset(LoadLibraryW(L"dinput8"));
if (!m_dinput_module)
{
Console.Error("Failed to load DInput module.");
return false;
}
PFNDIRECTINPUT8CREATE create = reinterpret_cast<PFNDIRECTINPUT8CREATE>(GetProcAddress(m_dinput_module, "DirectInput8Create"));
PFNGETDFDIJOYSTICK get_joystick_data_format = reinterpret_cast<PFNGETDFDIJOYSTICK>(GetProcAddress(m_dinput_module, "GetdfDIJoystick"));
PFNDIRECTINPUT8CREATE create = reinterpret_cast<PFNDIRECTINPUT8CREATE>(GetProcAddress(m_dinput_module.get(), "DirectInput8Create"));
PFNGETDFDIJOYSTICK get_joystick_data_format = reinterpret_cast<PFNGETDFDIJOYSTICK>(GetProcAddress(m_dinput_module.get(), "GetdfDIJoystick"));
if (!create || !get_joystick_data_format)
{
Console.Error("Failed to get DInput function pointers.");

11
pcsx2/Frontend/DInputSource.h
View File

@ -24,6 +24,8 @@
#include <mutex>
#include <vector>
#include <wil/resource.h>
class DInputSource final : public InputSource
{
public:
@ -83,10 +85,11 @@ private:
void CheckForStateChanges(size_t index, const DIJOYSTATE& new_state);
ControllerDataArray m_controllers;
HMODULE m_dinput_module{};
// Those must go first in the class so they are destroyed last
wil::unique_hmodule m_dinput_module;
wil::com_ptr_nothrow<IDirectInput8W> m_dinput;
LPCDIDATAFORMAT m_joystick_data_format{};
HWND m_toplevel_window = NULL;
HWND m_toplevel_window = nullptr;
ControllerDataArray m_controllers;
};