From 469ecd6b2967e98061bed1b30f77158373c08844 Mon Sep 17 00:00:00 2001 From: Jonny Lamb Date: Fri, 30 Jan 2009 17:35:18 +0000 Subject: Simplified loops in Simplified loops in contact_selector_get_{number_online_contacts,iter_for_blank_contact}. Signed-off-by: Jonny Lamb svn path=/trunk/; revision=2349 --- libempathy-gtk/empathy-contact-selector.c | 40 +++++++++++++++---------------- 1 file changed, 19 insertions(+), 21 deletions(-) (limited to 'libempathy-gtk') diff --git a/libempathy-gtk/empathy-contact-selector.c b/libempathy-gtk/empathy-contact-selector.c index f1a1a6873..e4648f241 100644 --- a/libempathy-gtk/empathy-contact-selector.c +++ b/libempathy-gtk/empathy-contact-selector.c @@ -56,17 +56,16 @@ contact_selector_get_number_online_contacts (GtkTreeStore *store) GtkTreeIter tmp_iter; gboolean is_online; guint number_online_contacts = 0; + gboolean ok; - if (gtk_tree_model_get_iter_first (GTK_TREE_MODEL (store), &tmp_iter)) + for (ok = gtk_tree_model_get_iter_first (GTK_TREE_MODEL (store), &tmp_iter); + ok; ok = gtk_tree_model_iter_next (GTK_TREE_MODEL (store), &tmp_iter)) { - do - { - gtk_tree_model_get (GTK_TREE_MODEL (store), - &tmp_iter, EMPATHY_CONTACT_LIST_STORE_COL_IS_ONLINE, - &is_online, -1); - if (is_online) - number_online_contacts++; - } while (gtk_tree_model_iter_next (GTK_TREE_MODEL (store), &tmp_iter)); + gtk_tree_model_get (GTK_TREE_MODEL (store), + &tmp_iter, EMPATHY_CONTACT_LIST_STORE_COL_IS_ONLINE, + &is_online, -1); + if (is_online) + number_online_contacts++; } return number_online_contacts; @@ -79,21 +78,20 @@ contact_selector_get_iter_for_blank_contact (GtkTreeStore *store, GtkTreeIter tmp_iter; EmpathyContact *tmp_contact; gboolean is_present = FALSE; + gboolean ok; - if (gtk_tree_model_get_iter_first (GTK_TREE_MODEL (store), &tmp_iter)) + for (ok = gtk_tree_model_get_iter_first (GTK_TREE_MODEL (store), &tmp_iter); + ok; ok = gtk_tree_model_iter_next (GTK_TREE_MODEL (store), &tmp_iter)) { - do + gtk_tree_model_get (GTK_TREE_MODEL (store), + &tmp_iter, EMPATHY_CONTACT_LIST_STORE_COL_CONTACT, + &tmp_contact, -1); + if (tmp_contact == NULL) { - gtk_tree_model_get (GTK_TREE_MODEL (store), - &tmp_iter, EMPATHY_CONTACT_LIST_STORE_COL_CONTACT, - &tmp_contact, -1); - if (tmp_contact == NULL) - { - *blank_iter = tmp_iter; - is_present = TRUE; - break; - } - } while (gtk_tree_model_iter_next (GTK_TREE_MODEL (store), &tmp_iter)); + *blank_iter = tmp_iter; + is_present = TRUE; + break; + } } return is_present; -- cgit v1.2.3