comparison src/gtkblist.c @ 10504:1a97d5e88d12

[gaim-migrate @ 11796] Lots of things here: - Several memory leak fixes - A few invalid memory access fixes - Fix a yahoo crash going idle when away - Fix Add user in chats to actually fill in the screenname - Add gaim_account_{get,set}_enabled to perl - Fix command priorities (fixes /me in IRC) - Fix MSN notification server transfer to be quiet about it - Fix MSN blist sync if user has insane friendly name - Make the docklet less crash-happy if it fails to embed in 3 seconds - Only probe for native plugins with the correct file extension - 1 typo fix :) ... and quite possibly something else I forgot. committer: Tailor Script <tailor@pidgin.im>
author Stu Tomlinson <stu@nosnilmot.com>
date Tue, 11 Jan 2005 17:25:06 +0000
parents 4806967e093e
children 1044c0930e03
comparison
equal deleted inserted replaced
10503:776586d647e3 10504:1a97d5e88d12
2225 return FALSE; 2225 return FALSE;
2226 gtk_tree_model_get_iter(GTK_TREE_MODEL(gtkblist->treemodel), &iter, path); 2226 gtk_tree_model_get_iter(GTK_TREE_MODEL(gtkblist->treemodel), &iter, path);
2227 gtk_tree_model_get_value (GTK_TREE_MODEL(gtkblist->treemodel), &iter, NODE_COLUMN, &val); 2227 gtk_tree_model_get_value (GTK_TREE_MODEL(gtkblist->treemodel), &iter, NODE_COLUMN, &val);
2228 node = g_value_get_pointer(&val); 2228 node = g_value_get_pointer(&val);
2229 2229
2230 if(!GAIM_BLIST_NODE_IS_CONTACT(node)) 2230 if(!GAIM_BLIST_NODE_IS_CONTACT(node)) {
2231 gtk_tree_path_free(path);
2231 return FALSE; 2232 return FALSE;
2233 }
2232 2234
2233 gtknode = node->ui_data; 2235 gtknode = node->ui_data;
2234 2236
2235 if (!gtknode->contact_expanded) { 2237 if (!gtknode->contact_expanded) {
2236 GtkTreeIter i; 2238 GtkTreeIter i;
2276 return FALSE; 2278 return FALSE;
2277 gtk_tree_model_get_iter(GTK_TREE_MODEL(gtkblist->treemodel), &iter, path); 2279 gtk_tree_model_get_iter(GTK_TREE_MODEL(gtkblist->treemodel), &iter, path);
2278 gtk_tree_model_get_value (GTK_TREE_MODEL(gtkblist->treemodel), &iter, NODE_COLUMN, &val); 2280 gtk_tree_model_get_value (GTK_TREE_MODEL(gtkblist->treemodel), &iter, NODE_COLUMN, &val);
2279 node = g_value_get_pointer(&val); 2281 node = g_value_get_pointer(&val);
2280 2282
2283 gtk_tree_path_free(path);
2284
2281 if(!GAIM_BLIST_NODE_IS_CONTACT(node) && !GAIM_BLIST_NODE_IS_BUDDY(node) 2285 if(!GAIM_BLIST_NODE_IS_CONTACT(node) && !GAIM_BLIST_NODE_IS_BUDDY(node)
2282 && !GAIM_BLIST_NODE_IS_CHAT(node)) 2286 && !GAIM_BLIST_NODE_IS_CHAT(node))
2283 return FALSE; 2287 return FALSE;
2284 2288
2285 gtknode = node->ui_data; 2289 gtknode = node->ui_data;
2286
2287 gtk_tree_path_free(path);
2288 2290
2289 tooltiptext = gaim_get_tooltip_text(node); 2291 tooltiptext = gaim_get_tooltip_text(node);
2290 2292
2291 if(!tooltiptext) 2293 if(!tooltiptext)
2292 return FALSE; 2294 return FALSE;
3632 gaim_gtk_blist_hide_node(list, node); 3634 gaim_gtk_blist_hide_node(list, node);
3633 3635
3634 if(node->parent) 3636 if(node->parent)
3635 gaim_gtk_blist_update(list, node->parent); 3637 gaim_gtk_blist_update(list, node->parent);
3636 3638
3637 /* There's something I don't understand here */ 3639 /* There's something I don't understand here - Ethan */
3638 /* g_free(node->ui_data); 3640 /* Ethan said that back in 2003, but this g_free has been left commented
3639 node->ui_data = NULL; */ 3641 * out ever since. I can't find any reason at all why this is bad and
3642 * valgrind found several reasons why it's good. If this causes problems
3643 * comment it out again. Stu */
3644 g_free(node->ui_data);
3645 node->ui_data = NULL;
3640 } 3646 }
3641 3647
3642 static gboolean do_selection_changed(GaimBlistNode *new_selection) 3648 static gboolean do_selection_changed(GaimBlistNode *new_selection)
3643 { 3649 {
3644 GaimBlistNode *old_selection = NULL; 3650 GaimBlistNode *old_selection = NULL;