comparison libpurple/protocols/jabber/roster.c @ 15786:e926951e61fe

Don't use g_list_length() and g_slist_length() when all you want to do if check if the list is empty. Those two functions needlessly iterate through the entire list.
author Mark Doliner <mark@kingant.net>
date Tue, 13 Mar 2007 07:19:06 +0000
parents b4ac51aebf65
children 32c366eeeb99
comparison
equal deleted inserted replaced
15785:eed84b59c252 15786:e926951e61fe
378 GaimGroup *group) { 378 GaimGroup *group) {
379 GSList *buddies = gaim_find_buddies(gc->account, buddy->name); 379 GSList *buddies = gaim_find_buddies(gc->account, buddy->name);
380 GSList *groups = NULL; 380 GSList *groups = NULL;
381 381
382 buddies = g_slist_remove(buddies, buddy); 382 buddies = g_slist_remove(buddies, buddy);
383 if(g_slist_length(buddies)) { 383 if(buddies != NULL) {
384 GaimBuddy *tmpbuddy; 384 GaimBuddy *tmpbuddy;
385 GaimGroup *tmpgroup; 385 GaimGroup *tmpgroup;
386 386
387 while(buddies) { 387 while(buddies) {
388 tmpbuddy = buddies->data; 388 tmpbuddy = buddies->data;