all repos — fluxbox @ ccba298cb59e2d2dbd2235188e0695fff62c4799

custom fork of the fluxbox windowmanager 

fix a bug with Some and Every commands and the stupid global current window variable
Mark Tiefenbruck mark@fluxbox.org
commit

ccba298cb59e2d2dbd2235188e0695fff62c4799

parent

19d024cc53817a767d661698c448dbc9f8d869ab

1 files changed, 6 insertions(+), 2 deletions(-)

jump to
M src/WorkspaceCmd.cc → src/WorkspaceCmd.cc 
@@ -125,8 +125,10 @@ for (; it != it_end; ++it) {
             WinClient *client = dynamic_cast<WinClient *>(*it);
             if (!client) continue;
             WindowCmd<void>::setClient(client);
-            if (m_cmd->execute())
+            if (m_cmd->execute()) {
+                WindowCmd<void>::setClient(old);
                 return true;
+            }
         }
         WindowCmd<void>::setClient(old);
     }

@@ -146,8 +148,10 @@ for (; it != it_end; ++it) {
             WinClient *client = dynamic_cast<WinClient *>(*it);
             if (!client) continue;
             WindowCmd<void>::setClient(client);
-            if (!m_cmd->execute())
+            if (!m_cmd->execute()) {
+                WindowCmd<void>::setClient(old);
                 return false;
+            }
         }
         WindowCmd<void>::setClient(old);
     }