-
-
Notifications
You must be signed in to change notification settings - Fork 540
Fix multiple game controllers interfering each other #3369
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
base: master
Are you sure you want to change the base?
Fix multiple game controllers interfering each other #3369
Conversation
Having two controllers connected and playing local multiplayer was really horrible as one player blocked the other very often. With these changes, this effect does not occur anymore. This removes the attempt to get axis and d-pad somehow kind of synchronized, but I do not really see making these changes it any worse, because it would be anyway hard to find an appropriate solution. If the button/stick state should be kept as it was, it should be added as a per controller state and not a global one. @MatusGuy : This reverts also the changes made in 68f4717. I was not sure about the original problem, maybe you can check if these changes either reintroduce the problem you had or maybe even fix it as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't test it as I don't have multiple game controllers. But code-wise everything is fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to have indeed reverted my changes. I'll see what I can do
Apply this patch and see if I haven't removed any functionality: |
I tested your patch. At a first glance it looked quite normal, but then I ran into a state that the second controller's stick was not able to move Tux to the right. With the D-Pad it was working, but with the stick it was not possible to move to the right (only to the left). This happend two times during a 2 minute test. I have not checked into detail what went wrong in this case and what steps exactly are needed to reproduce it. |
GameControllerManager shared one button/stick state between all connected game controllers. This led to one controller interfering with other controllers in multiplayer mode.
bb5df17
to
615b688
Compare
GameControllerManager shared one button/stick state between all connected game controllers. This led to one controller interfering with other controllers in multiplayer mode.