Mercurial > pidgin
changeset 5263:b476a30ef4e3
[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>
| author | Luke Schierer <lschiere@pidgin.im> |
|---|---|
| date | Wed, 30 Apr 2003 19:54:19 +0000 |
| parents | 799f9585988a |
| children | e4dadc60ea5a |
| files | src/gtkblist.c |
| diffstat | 1 files changed, 14 insertions(+), 3 deletions(-) [+] |
line wrap: on
line diff
--- 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)