[mutter] window-actor: Another simple clang warning fix



commit 878b1012b49eca0d5c4b6081b026277cede04e83
Author: Jasper St. Pierre <jstpierre mecheye net>
Date:   Fri Apr 27 00:17:41 2012 -0400

    window-actor: Another simple clang warning fix
    
    If we explicitly check for a NULL pointer, clang will assume
    that the pointer may be NULL at some point. We clearly rely
    on the pointer being non-NULL earlier, so fix this guy up.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=674876

 src/compositor/meta-window-actor.c |    7 ++-----
 1 files changed, 2 insertions(+), 5 deletions(-)
---
diff --git a/src/compositor/meta-window-actor.c b/src/compositor/meta-window-actor.c
index ef3c6d6..3751d34 100644
--- a/src/compositor/meta-window-actor.c
+++ b/src/compositor/meta-window-actor.c
@@ -745,11 +745,8 @@ meta_window_actor_has_shadow (MetaWindowActor *self)
    * Always put a shadow around windows with a frame - This should override
    * the restriction about not putting a shadow around ARGB windows.
    */
-  if (priv->window)
-    {
-      if (meta_window_get_frame (priv->window))
-        return TRUE;
-    }
+  if (meta_window_get_frame (priv->window))
+    return TRUE;
 
   /*
    * Do not add shadows to ARGB windows; eventually we should generate a



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]