make the execute process better

[chaz/openbox] / util / epist / epist.cc

diff --git a/util/epist/epist.cc b/util/epist/epist.cc
index f3084e8c9aecbbabb3146718eac9e2fb8b6bfe33..f1c577c2df12c126e0f6df9265d66bcdc27a8893 100644 (file)
--- a/util/epist/epist.cc
+++ b/util/epist/epist.cc
@@ -83,8 +83,8 @@ epist::epist(char **argv, char *dpy_name, char *rc_file)
     ::exit(1);
   }
 
-  addAction(Action::nextWorkspace, ControlMask, "Tab");
-  addAction(Action::prevWorkspace, ControlMask | ShiftMask, "Tab");
+  addAction(Action::nextScreen, ControlMask, "Tab");
+  addAction(Action::prevScreen, ControlMask | ShiftMask, "Tab");
   addAction(Action::nextWindow, Mod1Mask, "Tab");
   addAction(Action::prevWindow, Mod1Mask | ShiftMask, "Tab");
   addAction(Action::toggleshade, Mod1Mask, "F5");
@@ -92,6 +92,8 @@ epist::epist(char **argv, char *dpy_name, char *rc_file)
   addAction(Action::nextWindowOnAllWorkspaces, Mod1Mask | ControlMask, "Tab");
   addAction(Action::prevWindowOnAllWorkspaces,
             Mod1Mask | ShiftMask | ControlMask, "Tab");
+  addAction(Action::prevWorkspace, Mod1Mask, "Left");
+  addAction(Action::nextWorkspace, Mod1Mask, "Right");
   addAction(Action::raise, Mod1Mask, "Up");
   addAction(Action::lower, Mod1Mask, "Down");
   addAction(Action::moveWindowUp, Mod1Mask | ControlMask, "Up", 1);
@@ -204,6 +206,14 @@ XWindow *epist::findWindow(Window window) const {
 
 
 void epist::cycleScreen(int current, bool forward) const {
+  unsigned int i;
+  for (i = 0; i < _screens.size(); ++i)
+    if (_screens[i]->number() == current) {
+      current = i;
+      break;
+    }
+  assert(i < _screens.size());  // current is for an unmanaged screen
+  
   int dest = current + (forward ? 1 : -1);
 
   if (dest < 0) dest = (signed)_screens.size() - 1;
@@ -223,7 +233,7 @@ void epist::addAction(Action::ActionType act, unsigned int modifiers,
 
 
 void epist::addAction(Action::ActionType act, unsigned int modifiers,
-                      string key, std::string str) {
+                      string key, string str) {
   _actions.push_back(Action(act, XKeysymToKeycode(getXDisplay(),
                                                   XStringToKeysym(key.c_str())),
                             modifiers, str));