kc3-lang/SDL/src/haptic/windows
Branch :
Log
| Author | Commit | Date | CI | Message |
|---|---|---|---|---|
 |
6b64c47b | 2021-11-09 23:45:32 | haptics: Enumerate XInput/DInput joysticks after haptic init Since the haptic subsystem is usually initialized after the joystick subsystem, the initial calls to HapticMaybeAddDevice() from inside SDL_JoystickInit() will arrive too early to be handled by the haptic subsystem. We need to add those haptic devices for those already present joysticks ourselves. | |
|
715e7bda | 2021-11-09 01:43:44 | haptic: Don't enumerate joysticks if the joystick subsystem is active The joystick subsystem has complex precedence logic to deal multiple competing backends like XInput, RawInput, and WGI. Let it fire the MaybeAdd callbacks for joystick devices, since it knows which backend will end up managing them. This resolves a situation where the RawInput joystick backend would take control of an XInput device but the XInput haptic backend would still create a haptic device. Since the XInput joystick backend didn't own the underlying joystick device, we'd end up with an orphaned haptic device that didn't work with SDL_HapticOpenFromJoystick() on the associated joystick device. | |
 |
a6da2fbf | 2021-05-25 12:33:23 | Rename haptic methods to avoid confusion with joystick: `SDL_DINPUT_MaybeAddDevice`->`SDL_DINPUT_HapticMaybeAddDevice` `SDL_DINPUT_MaybeRemoveDevice`->`SDL_DINPUT_HapticMaybeRemoveDevice` `SDL_XINPUT_MaybeAddDevice`->`SDL_XINPUT_HapticMaybeAddDevice` `SDL_XINPUT_MaybeRemoveDevice`->`SDL_XINPUT_HapticMaybeRemoveDevice` | |
|
73dc68a7 | 2021-05-19 17:52:08 | Simplify and cleanup DirectInput joystick backend code: - Do not call IDirectInputDevice8_QueryInterface(device, &IID_IDirectInputDevice8,...) on DIRECTINPUTDEVICE8 device - Get joystick VendorID and ProductID via IDirectInputDevice8_GetProperty(.., DIPROP_VIDPID, ..) call instead of messing with DIDEVICEINSTANCE.guidProduct - Normalize HID device interface path to upper case for stable operation of XInput check - Remove useless RawInput calls in SDL_IsXInputDevice() - just check for "IG_" string in HID device interface path that we already have There shouldn't be any observable behavior changes. | |
 |
9130f7c3 | 2021-01-02 10:25:38 | Updated copyright for 2021 | |
|
cb361896 | 2020-12-09 07:16:22 | Fixed bug 5235 - All internal sources should include SDL_assert.h Ryan C. Gordon We should really stick this in SDL_internal.h or something so it's always available. | |
 |
b1626295 | 2020-07-08 17:28:34 | cmake: Fix building with -DSDL_HAPTIC=Off | |
|
efe09359 | 2020-06-15 10:31:16 | Fix compile without DIRECTX | |
 |
68777406 | 2020-05-20 16:58:33 | windows: Fix calls to CoCreateInstance() so last parameter is a LPVOID *. | |
|
65ad4534 | 2020-05-04 13:17:43 | Improvement for bug 3446 - The haptic API does not allow to select the direction axes meyraud705 I see how the documentation is confusing. I think that the choice of the axis is an implementation detail. The documentation should state the goal of this value, so I propose this wording: "Use this value to play an effect on the steering wheel axis. This provides better compatibility across platforms and devices as SDL will guess the correct axis." Value could even be renamed 'SDL_HAPTIC_STEERING_AXIS'. For Linux, sending an effect on the X axis with a Logitech wheel works. Others brands don't have driver for Linux as far as I know. | |
|
73799917 | 2020-03-16 13:28:38 | Fixed bug 3446 - The haptic API does not allow to select the direction axes Mathieu Laurendeau Consider a device supporting effects on multiple axes. There's currently no way to play effects against a single-axis direction. A device supporting effects against X and Y may not allow to play effects with a two-axis direction coordinate, even if one of the coordinates is null. My current (ugly) work around for this is to add a direction type SDL_HAPTIC_X_FORCE to play effects against a X-axis only direction (patch attached). This issue impacted two GIMX users using the following wheels: - Leo Bodnar SimSteering force feedback wheel - Accuforce direct drive wheel Playing constant/spring/damper effects against a X-axis direction worked well for the first wheel, but not for the second one. A better strategy seems to play the effects against the first axis reported by the DirectInput enumeration. This strategy also works with Logitech wheels (at least the DFGT). It's been more than a year that I have the latest patch (playing effects against the first axis only) in the GIMX software. It's being used by thousands of people, mostly for adapting their FFB wheel to the PS4. I had no report that proves this strategy to be wrong. | |
|
a8780c6a | 2020-01-16 20:49:25 | Updated copyright date for 2020 | |
|
5e13087b | 2019-01-04 22:01:14 | Updated copyright for 2019 | |
|
f1bc1c12 | 2018-08-22 17:44:28 | Fixed crash trying to open HIDAPI controller as a haptics device on Windows | |
|
e3cc5b2c | 2018-01-03 10:03:25 | Updated copyright for 2018 | |
|
0d011ec6 | 2017-08-28 00:22:23 | Renaming of guard header names to quiet -Wreserved-id-macro | |
|
d96419c9 | 2017-06-21 01:22:00 | xinput: use the full range of the haptic motors (thanks, Trent!). XInput goes from 0 to 64k; we were feeding it values in the range of 0 to 32k. Fixes Bugzilla #3002. | |
|
70c0400b | 2017-02-13 17:00:46 | windows: Try to unify all the GUID comparison code into a core helper function. There are likely several more I missed. | |
|
dd007e3f | 2017-01-18 11:58:16 | Fixed bug 3561 - Re-acquire device before playing effects if needed. Mathieu Laurendeau Check the result of IDirectInputEffect_SetParameters and re-acquire the device to solve concurrency issues. | |
|
45b774e3 | 2017-01-01 18:33:28 | Updated copyright for 2017 | |
 |
97aa5775 | 2016-11-16 22:08:51 | Fixed empty parameter list in signatures of internal functions. | |
|
27d4f099 | 2016-10-07 23:40:44 | Implemented SDL_GetHintBoolean() to make it easier to check boolean hints | |
|
a21a227a | 2016-10-04 03:50:28 | Fixed bug 3021 - HapticOpenFromJoystick() problems Joe Thompson With Direct Input device (MOMO Steering Wheel w/FF) with SDL 2.0.3, SDL_HapticOpenFromJoystick() would fail. (Can't set exclusive mode) Now with 2.0.4 rc1, SDL_HapticOpenFromJoystick() succeeds but the the returned SDL_Haptic* cannot be used. Calls to SDL_HapticNewEffect() fail with "Haptic error Unable to create effect" If SDL_HapticOpen() is used instead of HapticOpenFromJoystick(), the device is usable. Calls to HapticNewEffect() succeed with the exact same parameters as the previous failing call. I have attached a proposed patch for this issue. When using SDL_HapticOpenFromJoystick(), the original code did not (re)enumerate the axes. This returned a new haptic device with 0 axes. Later, when a new effect is created, SDL_SYS_SetDirection() would set the flags to include DIEFF_SPHERICAL, regardless of what the caller actually set. (see Line 566 in SDL_dinputhaptic.c). This would cause the SDL_HapticNewEffect() to fail (or interpret the coordinates incorreclty.) The patch moves the call to IDirectInputDevice8_EnumObjects() outside of the if() block so that the axes are (re)enumerated for the new haptic device. Note: For steering wheels it is common for the joystick to have multiple axes (ie steering, throttle, brake), but the haptic portion of the joystick usually only applies to steering. | |
|
fa0f4176 | 2016-10-01 14:48:18 | Fixed build warnings and errors | |
|
9b4db2b8 | 2016-04-12 18:11:36 | Patched to compile on various platforms. | |
|
c61675dc | 2016-04-12 16:45:10 | threads: Move SDL's own thread creation to a new internal API. This allows us to set an explicit stack size (overriding the system default and the global hint an app might have set), and remove all the macro salsa for dealing with _beginthreadex and such, as internal threads always set those to NULL anyhow. I've taken some guesses on reasonable (and tiny!) stack sizes for our internal threads, but some of these might turn out to be too small in practice and need an increase. Most of them are simple functions, though. | |
|
fa8c83c1 | 2016-01-03 06:50:50 | Remove almost all instances of "volatile" keyword. As Tiffany pointed out in Bugzilla, volatile is not useful for thread safety: https://software.intel.com/en-us/blogs/2007/11/30/volatile-almost-useless-for-multi-threaded-programming/ Some of these volatiles didn't need to be, some were otherwise protected by spinlocks or mutexes, and some got moved over to SDL_atomic_t data, etc. Fixes Bugzilla #3220. | |
|
42065e78 | 2016-01-02 10:10:34 | Updated copyright to 2016 | |
|
0e45984f | 2015-06-21 17:33:46 | Fixed crash if initialization of EGL failed but was tried again later. The internal function SDL_EGL_LoadLibrary() did not delete and remove a mostly uninitialized data structure if loading the library first failed. A later try to use EGL then skipped initialization and assumed it was previously successful because the data structure now already existed. This led to at least one crash in the internal function SDL_EGL_ChooseConfig() because a NULL pointer was dereferenced to make a call to eglBindAPI(). | |
|
e3df6d5e | 2015-06-14 19:21:13 | Fixed bug 2953 - Crash due to a bad cleanup in the SDL_SYS_HapticQuit function Technically this is caused by the haptic devices not being closed at quit time, which we need to fix anyway, but this is a bandaid for now. | |
|
8bac7967 | 2015-06-07 18:29:23 | Maybe patched to compile on some Windows configurations. (Maybe) Fixes Bugzilla #3001. | |
|
84ce0006 | 2015-05-28 01:27:24 | I think this will be the time... | |
|
7964f3ad | 2015-05-28 01:22:14 | Still trying to get this to compile... | |
|
cae4fd7f | 2015-05-28 01:16:55 | More patching to compile... | |
|
58447b24 | 2015-05-28 00:54:52 | Move tests from SDL_config higher up in Windows joystick/haptic code. Fixes Bugzilla #2932. | |
|
2c4a6ea0 | 2015-05-26 06:27:46 | Updated the copyright year to 2015 | |
|
b72938c8 | 2015-04-20 12:22:44 | Windows: Always set the system timer resolution to 1ms by default. An existing hint lets apps that don't need the timer resolution changed avoid this, to save battery, etc, but this fixes several problems in timing, audio callbacks not firing fast enough, etc. Fixes Bugzilla #2944. | |
 |
fe6c797c | 2015-04-10 23:30:31 | Fixed an iOS view orientation issue when SDL_GL_CreateContext or SDL_CreateRenderer is called. | |
 |
b88ca1b4 | 2015-02-10 16:28:56 | the last parameter of XChangeProperty is the number of elements.. and when the element format is 32.. the element is "long" so we have 5 long elements here. Yes this seems confusing as on mac+linux Long is either 32 or 64bits depending on the architecture, but this is how the X11 protocol is defined. Thus 5 is the correct value for the nelts here. Not 5 or 10 depending on the architecture. More info on the confusion https://bugs.freedesktop.org/show_bug.cgi?id=16802 | |
|
b48e54aa | 2015-01-26 22:00:29 | Fixed bug 2802 - [patch] Fix android build compiling in wrong filesystem implementation Jonas Kulla The configure script didn't differentiate between Linux and Android, unconditionally compiling in the unix implementation of SDL_sysfilesystem.c. I'm probably one of the very few people building SDL for android using classic configure + standalone toolchain, so this has gone undetected all along. | |
 |
70438be2 | 2014-12-03 10:55:23 | WinRT: fixed bug whereby SDL would override an app's default orientation WinRT apps can set a default, preferred orientation via a .appxmanifest file. SDL was overriding this on app startup, and making the app use all possible orientations (landscape and portrait). Thanks to Eric Wing for the heads up on this! | |
|
9c398852 | 2014-11-22 22:20:40 | Corrected header file documentation comment. | |
|
5f9ea7ed | 2014-09-17 14:49:36 | Haptic: Deal with negative periodic magnitudes (thanks, Elias!). A negative periodic magnitude doesn't exist in Windows' and MacOS' FF APIs The periodic magnitude parameter of the SDL Haptic API is based on the Linux FF API, so it means they are not directly compatible: 'dwMagnitude' is a 'DWORD', which is unsigned. Fixes Bugzilla #2701. | |
 |
24c86b55 | 2014-09-11 19:24:42 | [X11] Reconcile logical keyboard state with physical state on FocusIn since the window system doesn't do it for us like other platforms. This prevents sticky keys and missed keys when going in and out of focus, for example Alt would appear to stick if switching away from an SDL app with Alt-Tab and had to be pressed again. CR: Sam | |
|
e2bbc17d | 2014-08-16 16:49:00 | Haptic: Fix clamping bugs on Windows, by using the Darwin haptics code. Thanks, Elias! Partially fixes Bugzilla #2686. | |
|
57db2790 | 2014-08-16 16:47:42 | Haptic: Fix the saturation and deadband parameters' available range. There was a misconception that Linux's saturation and deadband parameters - on which the corresponding SDL parameters were based - use only half of the possible range. Thanks, Elias! Partially fixes Bugzilla #2686. | |
|
1ee96bb9 | 2014-07-07 10:26:28 | Fixed mingw64 build and warnings | |
|
b79e7f32 | 2014-07-03 15:39:55 | Split the XInput and DirectInput code so Windows RT can use the existing XInput support. | |
|
3dcb451f | 2014-04-09 21:29:19 | Added a README file regarding WinRT support To note, this file is currently formatted with CRLF line endings, rather than LF, to allow the file to be viewed with Notepad. | |
|
764aa147 | 2014-03-29 12:29:38 | Fixed XInput haptic support on Windows 8 It turns out the XBox 360 controller driver never reports force feedback capability, so we'll try to set 0 state and see if that succeeds. | |
|
18c9a4e9 | 2014-03-18 17:16:28 | Fixed SDL_HapticNewEffect() failing on various DirectInput devices. | |
|
7eaf8997 | 2014-03-17 19:11:18 | Fixed SDL_HapticOpenFromJoystick() with DirectInput devices. | |
|
1005c934 | 2014-03-02 00:02:56 | Windows: Fixed crash if quitting without closing an XInput haptic device. | |
 |
1084421c | 2014-02-24 10:25:02 | Zero out haptic linked list items on creation (thanks to Turo Lamminen for the report!) | |
|
7033e597 | 2014-02-06 21:26:41 | Fixed crash on Windows if haptic isn't initialized when controllers are added. | |
|
a09548eb | 2014-02-06 09:35:44 | Fix device counting in HapticMouse and JoystickOpen routines. 0 is the first item in the list not the last | |
|
e42d46b4 | 2014-02-06 07:37:20 | Wired up haptic hotplugging for Windows DirectInput/XInput code. | |
|
58edac3e | 2014-02-02 00:53:27 | Fixed bug 2374 - Update copyright for 2014... Is it that time already?? | |
|
090327e7 | 2013-12-09 16:03:18 | Implemented the Dynamic API magic. | |
|
f848adff | 2013-11-29 10:06:08 | Improve Android pause/resume behavior. | |
|
7e1289af | 2013-11-24 23:56:17 | Make internal SDL sources include SDL_internal.h instead of SDL_config.h The new header will include SDL_config.h, but allows for other global stuff. | |
|
965a0da9 | 2013-10-23 15:54:12 | Moved a SDL_SetError() call elsewhere to avoid triggering it needlessly. Otherwise, the XInput path would always trigger it in a harmless manner. | |
|
aa86e05d | 2013-10-20 20:49:36 | Fixed windows compile | |
|
f5fa492e | 2013-10-20 20:42:55 | Added a macro SDL_TICKS_PASSED() to correctly compare two 32-bit tick values. Went through the code and used the macro and fixed a couple places that were using incorrect timestamp comparisons. | |
|
d7103996 | 2013-10-20 23:38:19 | Fix some edge cases in XInput haptic timing. | |
|
f28c7fe8 | 2013-10-20 16:03:11 | Backed out hg changeset add2dbe99fe2. REFGUID is apparently a const type. | |
|
c521ae84 | 2013-10-20 16:02:24 | Patched to compile on non-C99 Microsoft compiler. | |
|
a614f186 | 2013-10-20 16:01:10 | Patched to compile on Windows. | |
|
b285b600 | 2013-10-20 15:55:47 | Allow XInput haptics to run for SDL_HAPTIC_INFINITY time (thanks, Mitchell!). Partially fixes Bugzilla #2126. | |
|
6e1169dd | 2013-10-20 15:49:52 | Fixed goofy logic in haptic device comparison code. Mitchell Keith Bloch did the research on this bug, and came up with a slightly different patch than this. Partially fixes Bugzilla #2126. | |
|
54dce3f5 | 2013-10-20 15:45:48 | Don't bother calling SDL_SYS_HapticEffectType() at all for XInput devices. | |
|
f79fc33a | 2013-08-29 08:29:21 | Christoph Mallon: Remove pointless if (x) before SDL_free(x) | |
|
79499898 | 2013-08-28 17:12:07 | Fixed comment typo. | |
|
1e49b1ed | 2013-08-21 09:47:10 | OCD fixes: Adds a space after /* (glory to regular expressions!) | |
|
695344d1 | 2013-08-21 09:43:09 | OCD fixes: Adds a space before */ | |
|
dad42067 | 2013-08-12 11:13:50 | Fixes #2022, do not resume on Android when surfaceChanged If the app is in landscape mode and the user presses the power button, a pause is followed immediately by a surfaceChanged event because the lock screen is shown in portrait mode. This triggers a "false" resume. So, we just pause and resume following the onWindowFocusChanged events. Also, wait for SDL_APP_WILLENTERBACKGROUND and SDL_APP_DIDENTERBACKGROUND before blocking the event pump. | |
|
1ad936eb | 2013-08-11 19:56:43 | Fixed bug 2027 - Full-screen appears to be broken - hang in SDL_DestroyWindow() Rainer Deyke I'm running Linux Mint 15 with the Cinnamon window manager. SDL_DestroyWindow consistently locks up for me when the window if fullscreen. |