X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=util%2Fepist%2Fscreen.cc;h=e7ebd64cf1d1afb06abd79d40c69b74b6c49a88e;hb=41a38bbdaf245f12e58f2609092bfbcf8fba6f78;hp=7ac5780b1c59f4ce62543e029fee4e6496d7608f;hpb=d856db568a5933555191a35778dfa9cf0f619f8f;p=chaz%2Fopenbox

diff --git a/util/epist/screen.cc b/util/epist/screen.cc
index 7ac5780b..e7ebd64c 100644
--- a/util/epist/screen.cc
+++ b/util/epist/screen.cc
@@ -444,8 +444,7 @@ void screen::updateActiveWindow() {
     }
   }
   _active = it;
-  if (it != end)
-    _last_active = it;
+  _last_active = it;
 
   /*  cout << "Active window is now: ";
       if (_active == _clients.end()) cout << "None\n";
@@ -475,6 +474,7 @@ void screen::cycleWindow(const bool forward, const int increment,
                          const bool allscreens, const bool alldesktops,
                          const bool sameclass, const string &cn) const {
   assert(_managed);
+  assert(increment > 0);
 
   if (_clients.empty()) return;
 
@@ -486,7 +486,7 @@ void screen::cycleWindow(const bool forward, const int increment,
     begin = _clients.begin(),
     end = _clients.end();
 
-  const XWindow *t;
+  const XWindow *t = 0;
   
   for (int x = 0; x < increment; ++x) {
     while (1) {
@@ -531,8 +531,10 @@ void screen::cycleWindow(const bool forward, const int increment,
 }
 
 
-void screen::cycleWorkspace(const bool forward, const int increment, const bool loop) const {
+void screen::cycleWorkspace(const bool forward, const int increment,
+                            const bool loop) const {
   assert(_managed);
+  assert(increment > 0);
 
   unsigned int destination = _active_desktop;