callisto-debian/gtk3
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

treeview: Don't try to optimize away selection_changed signal

Browse Source 
We get certain cases, in particular with SELECTION_MULTIPLE, where we
cannot figure out in advance of real_set_cursor() if the selection will
actually change.
This commit is contained in:
Benjamin Otte
2011-12-10 06:49:29 +01:00
parent ef4c4b7836
commit d4d2cc14d2
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gtk/gtktreeview.c
View File

@ -9105,7 +9105,7 @@ gtk_tree_view_row_deleted (GtkTreeModel *model,
else else
gtk_tree_view_real_set_cursor (tree_view, NULL, CLEAR_AND_SELECT | CURSOR_INVALID); gtk_tree_view_real_set_cursor (tree_view, NULL, CLEAR_AND_SELECT | CURSOR_INVALID);
} }
else if (selection_changed) if (selection_changed)
g_signal_emit_by_name (tree_view->priv->selection, "changed"); g_signal_emit_by_name (tree_view->priv->selection, "changed");
} }
@ -12908,11 +12908,9 @@ gtk_tree_view_real_collapse_row (GtkTreeView *tree_view,
_gtk_rbtree_remove (node->children); _gtk_rbtree_remove (node->children);
/* if we change the cursor, we also change the selection,
* so no need to emit selection-changed. */
if (cursor_changed) if (cursor_changed)
gtk_tree_view_real_set_cursor (tree_view, path, CLEAR_AND_SELECT | CURSOR_INVALID); gtk_tree_view_real_set_cursor (tree_view, path, CLEAR_AND_SELECT | CURSOR_INVALID);
else if (selection_changed) if (selection_changed)
g_signal_emit_by_name (tree_view->priv->selection, "changed"); g_signal_emit_by_name (tree_view->priv->selection, "changed");
if (animate) if (animate)