The function StartWindozeCycle() uses now the variable WShortKey
to store the shortcut key.

The code is the same, but now the hasModifier variable is calculated
in only one place.

Signed-off-by: Rodolfo García Peñas (kix) <k...@kix.es>
---
 src/cycling.c | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/cycling.c b/src/cycling.c
index 291a28a..2cba22b 100644
--- a/src/cycling.c
+++ b/src/cycling.c
@@ -79,10 +79,11 @@ static WWindow *change_focus_and_raise(WWindow *newFocused, 
WWindow *oldFocused,
        return oldFocused;
 }
 
-void StartWindozeCycle(WWindow * wwin, XEvent * event, Bool next, Bool 
class_only)
+void StartWindozeCycle(WWindow *wwin, XEvent *event, Bool next, Bool 
class_only)
 {
 
        XModifierKeymap *keymap        = NULL;
+       WShortKey binding;
        WSwitchPanel    *swpanel       = NULL;
        WScreen         *scr           = wScreenForRootWindow(event->xkey.root);
        KeyCode         leftKey        = XKeysymToKeycode(dpy, XK_Left);
@@ -108,16 +109,17 @@ void StartWindozeCycle(WWindow * wwin, XEvent * event, 
Bool next, Bool class_onl
 
        if (next) {
                if (class_only)
-                       hasModifier = (wKeyBindings[WKBD_GROUPNEXT].modifier != 
0);
+                       binding = wKeyBindings[WKBD_GROUPNEXT];
                else
-                       hasModifier = (wKeyBindings[WKBD_FOCUSNEXT].modifier != 
0);
+                       binding = wKeyBindings[WKBD_FOCUSNEXT];
        } else {
                if (class_only)
-                       hasModifier = (wKeyBindings[WKBD_GROUPPREV].modifier != 
0);
+                       binding = wKeyBindings[WKBD_GROUPPREV];
                else
-                       hasModifier = (wKeyBindings[WKBD_FOCUSPREV].modifier != 
0);
+                       binding = wKeyBindings[WKBD_FOCUSPREV];
        }
 
+       hasModifier = (binding.modifier != 0);
        if (hasModifier) {
                keymap = XGetModifierMapping(dpy);
                XGrabKeyboard(dpy, scr->root_win, False, GrabModeAsync, 
GrabModeAsync, CurrentTime);
-- 
1.8.4.rc3


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.

Reply via email to