*
* Creates a list of path of all selected rows. Additionally, if you are
* planning on modifying the model after calling this function, you may
- * want to convert the returned list into a list of #GtkTreeRowReferences.
+ * want to convert the returned list into a list of #GtkTreeRowReference<!-- -->s.
* To do this, you can use gtk_tree_row_reference_new_proxy().
*
* To free the return value, use:
return count;
}
+/* gtk_tree_selection_selected_foreach helper */
+static void
+model_changed (gpointer data)
+{
+ gboolean *stop = (gboolean *)data;
+
+ *stop = TRUE;
+}
+
/**
* gtk_tree_selection_selected_foreach:
* @selection: A #GtkTreeSelection.
GtkRBNode *node;
GtkTreeIter iter;
+ guint inserted_id, deleted_id, reordered_id;
+ gboolean stop = FALSE, has_next = FALSE, has_parent = FALSE;
+
g_return_if_fail (GTK_IS_TREE_SELECTION (selection));
g_return_if_fail (selection->tree_view != NULL);
g_return_if_fail (selection->tree_view->priv->model != NULL);
while (node->left != tree->nil)
node = node->left;
+ /* connect to signals to monitor changes in treemodel */
+ inserted_id = g_signal_connect_swapped (selection->tree_view->priv->model,
+ "row_inserted",
+ G_CALLBACK (model_changed),
+ &stop);
+ deleted_id = g_signal_connect_swapped (selection->tree_view->priv->model,
+ "row_deleted",
+ G_CALLBACK (model_changed),
+ &stop);
+ reordered_id = g_signal_connect_swapped (selection->tree_view->priv->model,
+ "rows_reordered",
+ G_CALLBACK (model_changed),
+ &stop);
+
/* find the node internally */
path = gtk_tree_path_new_first ();
gtk_tree_model_get_iter (selection->tree_view->priv->model,
{
if (GTK_RBNODE_FLAG_SET (node, GTK_RBNODE_IS_SELECTED))
(* func) (selection->tree_view->priv->model, path, &iter, data);
+
+ if (stop)
+ goto out;
+
if (node->children)
{
gboolean has_child;
tmp = iter;
has_child = gtk_tree_model_iter_children (selection->tree_view->priv->model, &iter, &tmp);
gtk_tree_path_append_index (path, 0);
- /* Sanity Check! */
- TREE_VIEW_INTERNAL_ASSERT_VOID (has_child);
+
+ /* we do the sanity check at the bottom of this function */
+ if (has_child)
+ goto out;
}
else
{
tree = tree->parent_tree;
if (tree == NULL)
{
- gtk_tree_path_free (path);
/* we've run out of tree */
/* We're done with this function */
- return;
+
+ goto out;
}
+
has_parent = gtk_tree_model_iter_parent (selection->tree_view->priv->model, &iter, &tmp_iter);
gtk_tree_path_up (path);
- /* Sanity check */
- TREE_VIEW_INTERNAL_ASSERT_VOID (has_parent);
+
+ /* we do the sanity check at the bottom of this function */
+ if (has_parent)
+ goto out;
}
}
while (!done);
}
}
while (TRUE);
+
+out:
+ if (path)
+ gtk_tree_path_free (path);
+
+ g_signal_handler_disconnect (selection->tree_view->priv->model,
+ inserted_id);
+ g_signal_handler_disconnect (selection->tree_view->priv->model,
+ deleted_id);
+ g_signal_handler_disconnect (selection->tree_view->priv->model,
+ reordered_id);
+
+ /* check if we have to spew a scary message */
+ if (has_next)
+ TREE_VIEW_INTERNAL_ASSERT_VOID (has_next);
+ if (has_parent)
+ TREE_VIEW_INTERNAL_ASSERT_VOID (has_parent);
+ if (stop)
+ g_warning
+ ("The model has been modified from within gtk_tree_selection_foreach.\n"
+ "This function is for observing the selections of the tree only. If\n"
+ "you are trying to get all selected items from the tree, try using\n"
+ "gtk_tree_selection_get_selected_rows instead.\n");
}
/**
if (gtk_tree_selection_real_select_node (selection, tree, node, TRUE))
{
dirty = TRUE;
+ if (selection->tree_view->priv->anchor)
+ gtk_tree_row_reference_free (selection->tree_view->priv->anchor);
+
selection->tree_view->priv->anchor =
gtk_tree_row_reference_new_proxy (G_OBJECT (selection->tree_view), selection->tree_view->priv->model, path);
}