]> Pileus Git - ~andy/gtk/commitdiff
a11y: fix uninitialized variables compiler warning
authorCosimo Cecchi <cosimoc@gnome.org>
Fri, 14 Dec 2012 02:52:51 +0000 (21:52 -0500)
committerCosimo Cecchi <cosimoc@gnome.org>
Fri, 14 Dec 2012 02:52:51 +0000 (21:52 -0500)
The code path where we update the tooltip text property doesn't set
the state and value variables, and so doesn't need to call
notify_state_change().
Return early, and move the if block at the beginning of the function for
clarity.

gtk/a11y/gtkwidgetaccessible.c

index 52d70391d8e13c5cf77e37376e80c953937f4a90..ebbcf3065d1afb22de3f63fc16f07393742dd2ab 100644 (file)
@@ -474,6 +474,12 @@ gtk_widget_accessible_notify_gtk (GObject    *obj,
      * focus changes so we ignore this.
      */
     return;
+  else if (g_strcmp0 (pspec->name, "tooltip-text") == 0)
+    {
+      gtk_widget_accessible_update_tooltip (GTK_WIDGET_ACCESSIBLE (atk_obj),
+                                            widget);
+      return;
+    }
   else if (g_strcmp0 (pspec->name, "visible") == 0)
     {
       state = ATK_STATE_VISIBLE;
@@ -494,11 +500,6 @@ gtk_widget_accessible_notify_gtk (GObject    *obj,
       state = ATK_STATE_HORIZONTAL;
       value = (gtk_orientable_get_orientation (orientable) == GTK_ORIENTATION_HORIZONTAL);
     }
-  else if (g_strcmp0 (pspec->name, "tooltip-text") == 0)
-    {
-      gtk_widget_accessible_update_tooltip (GTK_WIDGET_ACCESSIBLE (atk_obj),
-                                            widget);
-    }
   else
     return;