use g_[s]list_next/previous consistantly, and check for "it" instead of "it != NULL...

[chaz/openbox] / openbox / event.c

diff --git a/openbox/event.c b/openbox/event.c
index 700013b8d793911b619ea408a36dd9c567ef74ee..71dca5f7da3dc702b0c3507b840c1ac1c42320c3 100644 (file)
--- a/openbox/event.c
+++ b/openbox/event.c
@@ -846,7 +846,10 @@ static void event_handle_client(ObClient *client, XEvent *e)
     case MapRequest:
         ob_debug("MapRequest for 0x%lx\n", client->window);
         if (!client->iconic) break; /* this normally doesn't happen, but if it
-                                       does, we don't want it! */
+                                       does, we don't want it!
+                                       it can happen now when the window is on
+                                       another desktop, but we still don't
+                                       want it! */
         client_activate(client, FALSE);
         break;
     case ClientMessage: