Mercurial > pidgin.yaz
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; |