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

ControllerInterface: Remove unused "force" parameter 

I'm not sure when this nonsense of forcing locking the mutex when it's
already taken should have ever taken effect, but let's be thankful it
isn't now. That was a badly worded sentence.
This commit is contained in:
Jasper St. Pierre 2014-11-13 00:10:02 -08:00
parent 6aa1a59ee8
commit b5d4e8d37e
2 changed files with 6 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
Source/Core/InputCommon/ControllerInterface/ControllerInterface.cpp
View File

@ -135,13 +135,11 @@ void ControllerInterface::Shutdown()
//
// Update input for all devices, return true if all devices returned successful
//
bool ControllerInterface::UpdateInput(const bool force)
bool ControllerInterface::UpdateInput()
{
std::unique_lock<std::recursive_mutex> lk(update_lock, std::defer_lock);
if (force)
lk.lock();
else if (!lk.try_lock())
if (!lk.try_lock())
return false;
size_t ok_count = 0;
@ -163,13 +161,11 @@ bool ControllerInterface::UpdateInput(const bool force)
//
// Update output for all devices, return true if all devices returned successful
//
bool ControllerInterface::UpdateOutput(const bool force)
bool ControllerInterface::UpdateOutput()
{
std::unique_lock<std::recursive_mutex> lk(update_lock, std::defer_lock);
if (force)
lk.lock();
else if (!lk.try_lock())
if (!lk.try_lock())
return false;
size_t ok_count = 0;

4
Source/Core/InputCommon/ControllerInterface/ControllerInterface.h
View File

@ -120,8 +120,8 @@ public:
bool IsInit() const { return m_is_init; }
void UpdateReference(ControlReference* control, const ciface::Core::DeviceQualifier& default_device) const;
bool UpdateInput(const bool force = false);
bool UpdateOutput(const bool force = false);
bool UpdateInput();
bool UpdateOutput();
std::recursive_mutex update_lock;