all repos — openbox @ fa301a99952a74545a5b399f0227f91fa66c0ab3

openbox fork - make it a bit more like ryudo

Fixed a boob mistake which was causing KeyRelease events to not be handled properly if keys were relased quickly.
Marius Nita marius@cs.pdx.edu
commit

fa301a99952a74545a5b399f0227f91fa66c0ab3

parent

a95a6bd8e81a6bcc9db8abbbb3c0c78f087733f4

1 files changed, 1 insertions(+), 20 deletions(-)

jump to
M util/epist/screen.ccutil/epist/screen.cc

@@ -369,11 +369,7 @@ }

} -void screen::handleKeyrelease(const XEvent &e) { - // we're not interested in non-modifiers - if (!isModifier(e.xkey.keycode)) - return; - +void screen::handleKeyrelease(const XEvent &) { // the only keyrelease event we care about (for now) is when we do stacked // cycling and the modifier is released if (_stacked_cycling && _cycling && nothingIsPressed()) {

@@ -795,21 +791,6 @@ void screen::ungrabModifiers() const {

Display *display = _epist->getXDisplay(); XUngrabKeyboard(display, CurrentTime); -} - - -bool screen::isModifier(const KeyCode kc) const { - KeySym ks = XKeycodeToKeysym(_epist->getXDisplay(), kc, 0); - - if (ks == XK_Shift_L || ks == XK_Shift_R || - ks == XK_Control_L || ks == XK_Control_R || - ks == XK_Meta_L || ks == XK_Meta_R || - ks == XK_Alt_L || ks == XK_Alt_R || - ks == XK_Super_L || ks == XK_Super_R || - ks == XK_Hyper_L || ks == XK_Hyper_R) - return true; - else - return false; }