Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Re-enable Guide button on Windows 8/10 #288

Merged
merged 1 commit into from Aug 31, 2023
Merged

Conversation

garbear
Copy link
Member

@garbear garbear commented Aug 23, 2023

Description

This PR re-enables the Guide button on Windows 8/10. It was originally disabled in early 2017 in #79 due to the following crash reported by some users:

INVALID_POINTER_READ_DETOURS_c0000005_XInputUap.dll!XInputCore::XInputManager::_XInputManager

Now, we have some feature requests for the Guide button to return: https://forum.kodi.tv/showthread.php?tid=339917

So I figure we'll enable it for v21 and watch out for any crashes. If all goes well we can ship the Guide button in v21. If we hear more reports of crashes, we can look into other options.

How has this been tested?

Included in latest 20.2 test builds: https://github.com/garbear/xbmc/releases

Tested on latest master. Guide button works, but only after disabling the following setting:

Start -> Settings -> Gaming -> Xbox Game Bar

Disable "Open Xbox Game Bar using this button on a controller: X"

After that, the Guide button works in Kodi.

However, if Steam is open in the background, then it will be focused. Steam must be exited to not interfere with Kodi. There should be a way to fix this in Steam, however.

Initially, this was disabled in an attempt to fix the following crash:

  INVALID_POINTER_READ_DETOURS_c0000005_XInputUap.dll!XInputCore::XInputManager::_XInputManager
@garbear garbear merged commit a66f407 into xbmc:Omega Aug 31, 2023
6 checks passed
@garbear garbear deleted the enable-guide branch August 31, 2023 00:49
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.

None yet

1 participant