Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 499180 - media-libs/libsdl-1.2.15-r4 randomly looses KEYUP events if an IME is active
Summary: media-libs/libsdl-1.2.15-r4 randomly looses KEYUP events if an IME is active
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Games
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-25 01:00 UTC by Rafał Mużyło
Modified: 2015-05-12 19:57 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
a cargo-cult patch that seems to fix the problem (sdl-1.2-event_filter.patch,1.27 KB, patch)
2014-01-25 01:00 UTC, Rafał Mużyło
Details | Diff
patch corrected for an obvious problem (sdl-1.2-event_filter.patch,1.51 KB, patch)
2014-01-29 01:37 UTC, Rafał Mużyło
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Rafał Mużyło 2014-01-25 01:00:51 UTC
Created attachment 368658 [details, diff]
a cargo-cult patch that seems to fix the problem

Under certain conditions, SDL_PollEvents seems to be unexpectedly dropping KEYUP events if an IME is active.

The exact conditions are not clear - while it's related to SDL_EnableUNICODE flips, it doesn't happen every time. Both apps where I could trigger this problem (as an input freeze) are not in the tree.

The attached patch is a cargo-cult of a recent change in SDL2 made to a similar code block (https://hg.libsdl.org/SDL/rev/713c6a333c33).
I can't tell if this is a proper fix.

Upstream bug has been filed on 24.12, no response from upstream yet.

This is filed basically for *other* distros to take notice.

Perhaps someone from x11 herd could shed some light on specifics of interaction between XFilterEvent and xlib keyboard focus ?
Comment 1 Rafał Mużyło 2014-01-29 01:37:17 UTC
Created attachment 369020 [details, diff]
patch corrected for an obvious problem

As expected, the previous patch was somewhat broken in (in the hindsight) an obvious place.

That didn't affect the original problem, but broke other things.

This version has that new problem corrected.
Comment 2 Rafał Mużyło 2014-07-06 07:26:56 UTC
Upstream has accepted the patch...without any discussion. :sigh:

I can't say I'm really happy with that, but at least things seem to be working now.
Comment 3 Mr. Bones. (RETIRED) gentoo-dev 2015-05-12 19:57:19 UTC
added in libsdl-1.2.15-r9