ecore_x: fix key-up events!
authorbarbieri <barbieri@7cbeb6ba-43b4-40fd-8cce-4c39aea84d33>
Sun, 26 Sep 2010 19:41:02 +0000 (19:41 +0000)
committerbarbieri <barbieri@7cbeb6ba-43b4-40fd-8cce-4c39aea84d33>
Sun, 26 Sep 2010 19:41:02 +0000 (19:41 +0000)
commit9a610441cda274d29ebc36fc973307d0c5744384
tree06c63cd4ad1cf7ca5877e45437172bf754254dd6
parent74edb0636edf52f9134f583e7501cc8c0de1a738
ecore_x: fix key-up events!

Bad cedric, no cookie for you! While merging r39505 introducing
Ecore_Input you had all the code to go through Xutf8LookupString(),
but its documentation says (man Xutf8LookupString):

{{{
                                        Note

            ®To ensure proper input processing, it is essential that the
            client pass only KeyPress events to XmbLookupString,
            XwcLookupString and Xutf8LookupString.  Their behavior when a
            client passes a KeyRelease event is undefined.
}}}

Yeah, Xlib is quite stupid and this makes no sense.

As this just happens for UP events, it was unnoticed for a long time
(19 months) as most apps just handle DOWN events, as it gets X
keyboard repetition and all.

Thanks to Otavio Pontes that spotted this bug while doing some code
for EPhoto (that for some weird reason uses UP instead of DOWN
events).

git-svn-id: http://svn.enlightenment.org/svn/e/trunk/ecore@52786 7cbeb6ba-43b4-40fd-8cce-4c39aea84d33
src/lib/ecore_x/xlib/ecore_x_events.c