| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
Since C++17, the introduction of deduction guides for locking facilities
means that we no longer need to hardcode the mutex type into the locks
themselves, making it easier to switch mutex types, should it ever be
necessary in the future.
|
|
|
|
|
| |
In both cases, we weren't actually returning anything, which is
undefined behavior.
|
|
|
|
| |
Just makes the definitions a little bit more tidy.
|
|
|
|
|
| |
We don't need to universally capture by reference. We specifically just
need to capture the this pointer.
|
|
|
|
| |
Specifying the time unit itself is sufficient here.
|
| |
|
| |
|
|
|
|
| |
Any SDL invocation can call the even callback on the same thread, which can call GetSDLJoystickBySDLID and eventually cause double lock on joystick_map_mutex. To avoid this, lock guard should be placed as closer as possible to the object accessing code, so that any SDL invocation is with the mutex unlocked
|
|
|
|
|
|
|
|
|
| |
Changes the interface as well to remove any unique methods that
frontends needed to call such as StartJoystickEventHandler by
conditionally starting the polling thread only if the frontend hasn't
started it already. Additionally, moves all global state into a single
SDLState class in order to guarantee that the destructors are called in
the proper order
|
|
This should make reviewing much easier as you can then see what changed
happened between the old file and the new one
|