diff options
author | Matthias Clasen <mclasen@redhat.com> | 2022-05-06 21:36:22 -0400 |
---|---|---|
committer | Matthias Clasen <mclasen@redhat.com> | 2022-05-07 10:31:37 -0400 |
commit | d91c612eab83d3d989304965b84b8daa746c1be1 (patch) | |
tree | 8bd795511579c45195ec4bdf63a9961ecdc5268c | |
parent | 3894f04d0eb03dbba2814370c76402361345b448 (diff) | |
download | gtk+-d91c612eab83d3d989304965b84b8daa746c1be1.tar.gz |
listitemwidget: Avoid some unnecessary work
Only update widget and accessible state if the
selected property actually changed.
-rw-r--r-- | gtk/gtklistitemwidget.c | 25 |
1 files changed, 16 insertions, 9 deletions
diff --git a/gtk/gtklistitemwidget.c b/gtk/gtklistitemwidget.c index 56f807cf70..959d56fb4d 100644 --- a/gtk/gtklistitemwidget.c +++ b/gtk/gtklistitemwidget.c @@ -485,20 +485,27 @@ gtk_list_item_widget_update (GtkListItemWidget *self, gboolean selected) { GtkListItemWidgetPrivate *priv = gtk_list_item_widget_get_instance_private (self); + gboolean was_selected; + + was_selected = priv->selected; if (priv->list_item) gtk_list_item_factory_update (priv->factory, self, position, item, selected); else gtk_list_item_widget_default_update (self, NULL, position, item, selected); - if (selected) - gtk_widget_set_state_flags (GTK_WIDGET (self), GTK_STATE_FLAG_SELECTED, FALSE); - else - gtk_widget_unset_state_flags (GTK_WIDGET (self), GTK_STATE_FLAG_SELECTED); - - gtk_accessible_update_state (GTK_ACCESSIBLE (self), - GTK_ACCESSIBLE_STATE_SELECTED, selected, - -1); + /* don't look at selected variable, it's not reentrancy safe */ + if (was_selected != priv->selected) + { + if (priv->selected) + gtk_widget_set_state_flags (GTK_WIDGET (self), GTK_STATE_FLAG_SELECTED, FALSE); + else + gtk_widget_unset_state_flags (GTK_WIDGET (self), GTK_STATE_FLAG_SELECTED); + + gtk_accessible_update_state (GTK_ACCESSIBLE (self), + GTK_ACCESSIBLE_STATE_SELECTED, priv->selected, + -1); + } } void @@ -559,7 +566,7 @@ gtk_list_item_widget_default_update (GtkListItemWidget *self, GtkListItemWidgetPrivate *priv = gtk_list_item_widget_get_instance_private (self); /* FIXME: It's kinda evil to notify external objects from here... */ - + if (g_set_object (&priv->item, item)) notify_item = TRUE; |