# HG changeset patch
# User Luke Schierer <lschiere@pidgin.im>
# Date 1051732459 0
# Node ID b476a30ef4e38c96dfdc7f0b643848247b99e735
# Parent  799f9585988a7f6e077908583a265dda69323b99
[gaim-migrate @ 5635]
(15:53:01) Paco-Paco: It does not seem safe to remove ui_data on gaim_gtk_blist_remove, this patch disables that until I can figure out what I should _really_ be doing

committer: Tailor Script <tailor@pidgin.im>

diff -r 799f9585988a -r b476a30ef4e3 src/gtkblist.c
--- a/src/gtkblist.c	Wed Apr 30 04:49:09 2003 +0000
+++ b/src/gtkblist.c	Wed Apr 30 19:54:19 2003 +0000
@@ -1495,7 +1495,17 @@
 	struct _gaim_gtk_blist_node *gtknode = (struct _gaim_gtk_blist_node *)node->ui_data;
 	GtkTreePath *path;
 
-	if (!gtkblist || !gtknode->row)
+	if (!gtknode) {
+		gaim_debug(GAIM_DEBUG_ERROR, "gtkblist", "buddy %s has no ui_data\n", ((struct buddy *)node)->name);
+		return FALSE;
+	}
+
+	if (!gtkblist) {
+		gaim_debug(GAIM_DEBUG_ERROR, "gtkblist", "get_iter_from_node was called, but we don't seem to have a blist\n");
+		return FALSE;
+	}
+
+	if (!gtknode->row)
 		return FALSE;
 
 	if ((path = gtk_tree_row_reference_get_path(gtknode->row)) == NULL)
@@ -1550,8 +1560,9 @@
 {
 	gaim_gtk_blist_hide_node(list, node);
 
-	g_free(node->ui_data);
-	node->ui_data = NULL;
+	/* There's something I don't understand here */
+	/* g_free(node->ui_data);
+	node->ui_data = NULL; */
 }
 
 static gboolean do_selection_changed(GaimBlistNode *new_selection)