Skip to content

Fallback for non-gamepad controllers in GameInput#15187

Merged
slouken merged 1 commit into
libsdl-org:mainfrom
Nintorch:gameinput-ignore-non-gamepads
Mar 10, 2026
Merged

Fallback for non-gamepad controllers in GameInput#15187
slouken merged 1 commit into
libsdl-org:mainfrom
Nintorch:gameinput-ignore-non-gamepads

Conversation

@Nintorch
Copy link
Copy Markdown
Contributor

Description

This PR makes the GameInput joystick backend ignore non-gamepad controllers if DirectInput or XInput backends are enabled. This is done to prevent possible bugs and/or regressions. See also #15173 (comment) .

Existing Issue(s)

I couldn't find any.

@Nintorch
Copy link
Copy Markdown
Contributor Author

Although I should probably remove #else in case people might want to disable DInput/XInput via environment variables or hints

@slouken
Copy link
Copy Markdown
Collaborator

slouken commented Mar 10, 2026

Although I should probably remove #else in case people might want to disable DInput/XInput via environment variables or hints

Yep

This PR makes the GameInput joystick backend ignore non-gamepad controllers if DirectInput or XInput backends are enabled. This is done to prevent possible bugs and/or regressions.
@Nintorch Nintorch force-pushed the gameinput-ignore-non-gamepads branch from f36d1d1 to af0d9cd Compare March 10, 2026 17:51
@slouken slouken merged commit 419dcfe into libsdl-org:main Mar 10, 2026
44 of 45 checks passed
@slouken
Copy link
Copy Markdown
Collaborator

slouken commented Mar 10, 2026

Merged, thanks!

@Nintorch Nintorch deleted the gameinput-ignore-non-gamepads branch March 10, 2026 18:06
@slouken slouken added this to the 3.4.4 milestone Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants