GtkActionHelper: Remove unnecessary NULL check
authorDebarshi Ray <debarshir@gnome.org>
Tue, 13 Jun 2017 09:06:50 +0000 (11:06 +0200)
committerDebarshi Ray <debarshir@gnome.org>
Tue, 13 Jun 2017 19:58:18 +0000 (21:58 +0200)
This NULL check has existed since commit 652f16dd985dbc1b, when
GtkActionHelper was first introduced, but so has the assertion at the
top to ensure the validity of 'widget'.

https://bugzilla.gnome.org/show_bug.cgi?id=783587

gtk/gtkactionhelper.c

index 242528c921fdcd3e28b117cd544fd34d394ca203..41d3dd9cbd5615fd35f881c317f35549e04f5579 100644 (file)
@@ -387,22 +387,17 @@ GtkActionHelper *
 gtk_action_helper_new (GtkActionable *widget)
 {
   GtkActionHelper *helper;
+  GParamSpec *pspec;
 
   g_return_val_if_fail (GTK_IS_ACTIONABLE (widget), NULL);
   helper = g_object_new (GTK_TYPE_ACTION_HELPER, NULL);
 
   helper->widget = GTK_WIDGET (widget);
+  helper->enabled = gtk_widget_get_sensitive (GTK_WIDGET (helper->widget));
 
-  if (helper->widget)
-    {
-      GParamSpec *pspec;
-
-      helper->enabled = gtk_widget_get_sensitive (GTK_WIDGET (helper->widget));
-
-      pspec = g_object_class_find_property (G_OBJECT_GET_CLASS (helper->widget), "active");
-      if (pspec && G_PARAM_SPEC_VALUE_TYPE (pspec) == G_TYPE_BOOLEAN)
-        g_object_get (G_OBJECT (helper->widget), "active", &helper->active, NULL);
-    }
+  pspec = g_object_class_find_property (G_OBJECT_GET_CLASS (helper->widget), "active");
+  if (pspec && G_PARAM_SPEC_VALUE_TYPE (pspec) == G_TYPE_BOOLEAN)
+    g_object_get (G_OBJECT (helper->widget), "active", &helper->active, NULL);
 
   helper->action_context = _gtk_widget_get_action_muxer (GTK_WIDGET (widget), TRUE);