bugfix for SendToPrevWorkspace/TakeToPrevWorkspace, destination was calculated

wrong due to (wrong) implicit casting effects

1 files changed, 15 insertions, 11 deletions

diff --git a/src/CurrentWindowCmd.cc b/src/CurrentWindowCmd.cc
index 4a60e63..58ec78c 100644
--- a/src/CurrentWindowCmd.cc
+++ b/src/CurrentWindowCmd.cc
@@ -61,16 +61,18 @@ void SendToWorkspaceCmd::real_execute() {
 
 void SendToNextWorkspaceCmd::real_execute() {
     const int ws_nr =
-        ( fbwindow().workspaceNumber() + m_workspace_num ) %
+        ( fbwindow().workspaceNumber() + m_delta ) %
           fbwindow().screen().numberOfWorkspaces();
     fbwindow().screen().sendToWorkspace(ws_nr, &fbwindow(), false);
 }
 
 void SendToPrevWorkspaceCmd::real_execute() {
-    int ws_nr = (fbwindow().workspaceNumber() - m_workspace_num) %
-        fbwindow().screen().numberOfWorkspaces();
+    int ws_nr = (fbwindow().workspaceNumber() - m_delta );
     if ( ws_nr < 0 )
         ws_nr += fbwindow().screen().numberOfWorkspaces();
+
+    ws_nr = ws_nr % fbwindow().screen().numberOfWorkspaces();
+
     fbwindow().screen().sendToWorkspace(ws_nr, &fbwindow(), false);
 }
 
@@ -79,18 +81,20 @@ void TakeToWorkspaceCmd::real_execute() {
 }
 
 void TakeToNextWorkspaceCmd::real_execute() {
-    unsigned int workspace_num=
-        ( fbwindow().workspaceNumber() + m_workspace_num ) %
+    unsigned int ws_nr =
+        ( fbwindow().workspaceNumber() + m_delta) %
           fbwindow().screen().numberOfWorkspaces();
-    fbwindow().screen().sendToWorkspace(workspace_num, &fbwindow());
+    fbwindow().screen().sendToWorkspace(ws_nr, &fbwindow());
 }
 
 void TakeToPrevWorkspaceCmd::real_execute() {
-    int workspace_num = (fbwindow().workspaceNumber() - m_workspace_num) %
-        fbwindow().screen().numberOfWorkspaces();
-    if ( workspace_num < 0 )
-        workspace_num += fbwindow().screen().numberOfWorkspaces();
-    fbwindow().screen().sendToWorkspace(workspace_num, &fbwindow());
+    int ws_nr = (fbwindow().workspaceNumber() - m_delta);
+    if ( ws_nr < 0 )
+        ws_nr += fbwindow().screen().numberOfWorkspaces();
+
+    ws_nr = ws_nr % fbwindow().screen().numberOfWorkspaces();
+
+    fbwindow().screen().sendToWorkspace(ws_nr, &fbwindow());
 }
 
 void GoToTabCmd::real_execute() {