[Bug 55849] New: Wine sends WM_(SYS)KEYUP event twice for Alt (and such) key if Fcitx/iBus daemon is running
https://bugs.winehq.org/show_bug.cgi?id=55849 Bug ID: 55849 Summary: Wine sends WM_(SYS)KEYUP event twice for Alt (and such) key if Fcitx/iBus daemon is running Product: Wine Version: 8.19 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: winex11.drv Assignee: wine-bugs(a)winehq.org Reporter: emuser20140816(a)gmail.com Distribution: --- Created attachment 75355 --> https://bugs.winehq.org/attachment.cgi?id=75355 A test application I've attached an example application that reproduces this issue (win32-syskey-test.exe). The expected behavior of this application is to toggle background color (red/green) by pressing Alt (VK_MENU) key. This application behaves correctly on Windows, and behaves correctly on Wine too but only when the Fcitx/iBus daemon (with XIM integration enabled) is *not* running: WM_SYSKEYDOWN: wParam: 18, lParam: 0x20380001 WM_SYSKEYUP: wParam: 18, lParam: 0xc0380001 However, the application behaves differently on Wine if Fcitx/iBus daemon is running: WM_SYSKEYDOWN: wParam: 18, lParam: 0x20380001 WM_SYSKEYUP: wParam: 18, lParam: 0xc0380001 WM_KEYUP: wParam: 18, lParam: 0xc0380001 and the toggle switch no longer works. (This affects every application that takes a similar approach, i.e. an application that uses Alt, Ctrl or Shift key to toggle something on WM_(SYS)KEYUP event.) The workaround for this issue is either quit the Fcitx/iBus daemon (which makes IME unusable for this application) or revert the commit https://gitlab.winehq.org/wine/wine/-/commit/a4a5a2ec121eeb2a7cd4ffbb5ba7b29... (which requires complete rebuild from wine source tree). -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=55849 emuser20140816(a)gmail.com changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Wine sends WM_(SYS)KEYUP |Wine posts WM_(SYS)KEYUP |event twice for Alt (and |message twice for Alt (and |such) key if Fcitx/iBus |such) key if Fcitx/IBus |daemon is running |daemon is running -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=55849 glej0rlc <emuser20140816(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #75355|0 |1 is obsolete| | --- Comment #1 from glej0rlc <emuser20140816(a)gmail.com> --- Created attachment 75359 --> https://bugs.winehq.org/attachment.cgi?id=75359 A test application (revised) -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=55849 glej0rlc <emuser20140816(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1| |a4a5a2ec121eeb2a7cd4ffbb5ba | |7b298f2a050b1 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=55849 Olivier F. R. Dierick <o.dierick(a)piezo-forte.be> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |o.dierick(a)piezo-forte.be Keywords| |regression, testcase -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=55849 --- Comment #2 from Sven Baars <sven.wine@gmail.com> --- The behavior described here is basically just a description of the commit that was blamed. The only proper way to fix the issue described here is to revert that commit. Although I guess we could also make the workaround implemented in that commit more targeted. I think the right approach to fixing the original issue that was resolved by that commit would have actually been to submit a patch to SCIM, not to work around their issue in wine. But now that the patch has been committed, it would require some convincing to revert it again. For that reason, do you have any real world applications that are affected? The more real applications, the more likely it is that the patch can be reverted. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=55849 glej0rlc <emuser20140816@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|8.19 |11.0-rc3 Severity|normal |minor --- Comment #3 from glej0rlc <emuser20140816@gmail.com> --- I've ran into this issue in the game software that uses Alt key to release the mouse pointer from the game window. I can't think of any other cases. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=55849 Sven Baars <sven.wine@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|UNCONFIRMED |RESOLVED Fixed by SHA1| |5be4f61c4deade1228f045f6280 | |729132ffdd688 --- Comment #4 from Sven Baars <sven.wine@gmail.com> --- Should be fixed by 5be4f61c4deade1228f045f6280729132ffdd688. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=55849 Alexandre Julliard <julliard@winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #5 from Alexandre Julliard <julliard@winehq.org> --- Closing bugs fixed in 11.0-rc5. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (2)
-
WineHQ Bugzilla -
WineHQ Bugzilla