From: Dana Jansens <danakj@orodu.net>
Date: Tue, 3 Jun 2003 01:54:31 +0000 (+0000)
Subject: rm g_messages.. ones that segfault.. :>
X-Git-Url: https://git.brokenzipper.com/gitweb?a=commitdiff_plain;h=f7b687487cf26042ac79f2587fb1eaf1760a7a0b;p=chaz%2Fopenbox

rm g_messages.. ones that segfault.. :>
---

diff --git a/openbox/client.c b/openbox/client.c
index cf1acc97..d1890956 100644
--- a/openbox/client.c
+++ b/openbox/client.c
@@ -260,11 +260,12 @@ void client_manage(Window window)
         if (self->group) {
             GSList *it;
 
-            for (it = self->group->members; it; it = it->next)
+            for (it = self->group->members; it; it = it->next) {
                 if (client_focused(it->data)) {
                     group_foc = TRUE;
                     break;
                 }
+            }
         }
         /* note the check against Type_Normal/Dialog, not client_normal(self),
            which would also include other types. in this case we want more
@@ -1139,18 +1140,20 @@ void client_update_wmhints(Client *self)
                 group_remove(self->group, self);
                 self->group = NULL;
             }
-            /* i can only have transients from the group if i am not transient
-               myself */
-            if (hints->window_group != None && !self->transient_for) {
+            if (hints->window_group != None) {
                 self->group = group_add(hints->window_group, self);
 
-                /* add other transients of the group that are already
-                   set up */
-                for (it = self->group->members; it; it = it->next)
-                    if (it->data != self &&
-                        ((Client*)it->data)->transient_for == TRAN_GROUP)
-                        self->transients = g_slist_append(self->transients,
-                                                          it->data);
+                /* i can only have transients from the group if i am not
+                   transient myself */
+                if (!self->transient_for) {
+                    /* add other transients of the group that are already
+                       set up */
+                    for (it = self->group->members; it; it = it->next)
+                        if (it->data != self &&
+                            ((Client*)it->data)->transient_for == TRAN_GROUP)
+                            self->transients = g_slist_append(self->transients,
+                                                              it->data);
+                }
             }
 
             /* the WM_HINTS can contain an icon */