From: Dana Jansens <danakj@orodu.net>
Date: Sun, 13 Apr 2003 23:24:18 +0000 (+0000)
Subject: simple cleanups for the client_iconify function.
X-Git-Url: https://git.brokenzipper.com/gitweb?a=commitdiff_plain;h=e75ead70a6d98bd7e368c5844c05d2f85f275f65;p=chaz%2Fopenbox

simple cleanups for the client_iconify function.
also dont follow into other groups when searching up the transient_for tree
---

diff --git a/openbox/client.c b/openbox/client.c
index 055558e2..4b65b0d0 100644
--- a/openbox/client.c
+++ b/openbox/client.c
@@ -1598,6 +1598,8 @@ void client_fullscreen(Client *self, gboolean fs, gboolean savearea)
 
 void client_iconify(Client *self, gboolean iconic, gboolean curdesk)
 {
+    GSList *it;
+
     /* move up the transient chain as far as possible first if deiconifying */
     if (!iconic)
         while (self->transient_for) {
@@ -1615,8 +1617,7 @@ void client_iconify(Client *self, gboolean iconic, gboolean curdesk)
                     Client *c = it->data;
 
                     if (c != self && c->transient_for->iconic != iconic &&
-                        (c->transient_for != TRAN_GROUP ||
-                         c->group != self->group)) {
+                        c->transient_for != TRAN_GROUP) {
                         self = it->data;
                         break;
                     }
@@ -1652,12 +1653,8 @@ void client_iconify(Client *self, gboolean iconic, gboolean curdesk)
                     self, 0, 0);
 
     /* iconify all transients */
-    if (self->transients) {
-        GSList *it;
-
-        for (it = self->transients; it != NULL; it = it->next)
-            if (it->data != self) client_iconify(it->data, iconic, curdesk);
-    }
+    for (it = self->transients; it != NULL; it = it->next)
+        if (it->data != self) client_iconify(it->data, iconic, curdesk);
 }
 
 void client_maximize(Client *self, gboolean max, int dir, gboolean savearea)