-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Simplify the touch d-pad code. #18028
Conversation
Due to #17950 I had a look at this code and found some room to simplify, but I don't think this is likely to fix anything.
if (pressed & dir[i]) { | ||
if (g_Config.bHapticFeedback) { | ||
System_Vibrate(HAPTIC_VIRTUAL_KEY); | ||
__CtrlUpdateButtons(ctrlMask, CTRL_LEFT | CTRL_RIGHT | CTRL_UP | CTRL_DOWN); |
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 guess this makes the assumption that you would never use two inputs at once that affect the dpad. Before it was careful to only release things that were changed, but now any touch to the d-pad will FORCE release any other directions.
So if you have a combo key that presses say, up and triangle for some reason - and then you press left, the up button will be force released and discarded.
Is that intentional? I don't know if any games might motivate that sort of combo button, but it doesn't seem impossible at all.
-[Unknown]
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.
Oh, hm, I overlookee that. Might change it back
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.
Noticed I forgot to fix this. Fix going into 1.16.1
Due to #17950 I had a look at this code and found some room to simplify (now we never do more than one __CtrlUpdateButtons call), but I don't think this is likely to fix anything.