From: Christophe CURIS <[email protected]>

it is a bad habit to rely on a default value to make the work right, it is
safer to keep the assigned value in the same place of the code so a future
reviewer would not have to search for stuff (although the present case is
trivial).

(merge with 996dd1e35b0bf9055ba4b81c2cddc980cefafbc0)
---
 src/event.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/event.c b/src/event.c
index b839edc..86e2267 100644
--- a/src/event.c
+++ b/src/event.c
@@ -705,10 +705,12 @@ static void handleExpose(XEvent * event)
 static void executeWheelAction(WScreen *scr, XEvent *event, int action)
 {
        WWindow *wwin;
-       Bool next_direction = True;
+       Bool next_direction;
 
        if (event->xbutton.button == Button5 || event->xbutton.button == 
Button6)
                next_direction = False;
+       else
+               next_direction = True;
 
        switch (action) {
        case WA_SWITCH_WORKSPACES:
-- 
2.1.1


-- 
To unsubscribe, send mail to [email protected].

Reply via email to