From 8d377a95ed24080b068ceffc3cb2d6d12ccb15e4 Mon Sep 17 00:00:00 2001 From: Xavier Claessens Date: Wed, 26 Sep 2007 13:45:25 +0000 Subject: Fix warning when sort criterium is NULL and when there is no profile. 2007-09-26 Xavier Claessens * libempathy-gtk/empathy-profile-chooser.c: * libempathy-gtk/empathy-main-window.c: Fix warning when sort criterium is NULL and when there is no profile. Fixes bug #479786 (Christian Persch, Xavier Claessens). svn path=/trunk/; revision=318 --- libempathy-gtk/empathy-main-window.c | 6 +++--- libempathy-gtk/empathy-profile-chooser.c | 35 +++++++++++++++++--------------- 2 files changed, 22 insertions(+), 19 deletions(-) (limited to 'libempathy-gtk') diff --git a/libempathy-gtk/empathy-main-window.c b/libempathy-gtk/empathy-main-window.c index c021df7dd..7867b949c 100644 --- a/libempathy-gtk/empathy-main-window.c +++ b/libempathy-gtk/empathy-main-window.c @@ -897,13 +897,13 @@ main_window_notify_compact_contact_list_cb (EmpathyConf *conf, } static void -main_window_notify_sort_criterium_cb (EmpathyConf *conf, +main_window_notify_sort_criterium_cb (EmpathyConf *conf, const gchar *key, EmpathyMainWindow *window) { gchar *str = NULL; - if (empathy_conf_get_string (conf, key, &str)) { + if (empathy_conf_get_string (conf, key, &str) && str) { GType type; GEnumClass *enum_class; GEnumValue *enum_value; @@ -915,7 +915,7 @@ main_window_notify_sort_criterium_cb (EmpathyConf *conf, if (enum_value) { empathy_contact_list_store_set_sort_criterium (window->list_store, - enum_value->value); + enum_value->value); } } } diff --git a/libempathy-gtk/empathy-profile-chooser.c b/libempathy-gtk/empathy-profile-chooser.c index 4bc1d6c95..62c3b0f83 100644 --- a/libempathy-gtk/empathy-profile-chooser.c +++ b/libempathy-gtk/empathy-profile-chooser.c @@ -114,6 +114,7 @@ empathy_profile_chooser_new (void) GtkCellRenderer *renderer; GtkWidget *combo_box; GtkTreeIter iter; + gboolean iter_set = FALSE; /* set up combo box with new store */ store = gtk_list_store_new (COL_COUNT, @@ -136,15 +137,6 @@ empathy_profile_chooser_new (void) "text", COL_LABEL, NULL); - /* Set the profile sort function */ - gtk_tree_sortable_set_sort_column_id (GTK_TREE_SORTABLE (store), - COL_PROFILE, - GTK_SORT_ASCENDING); - gtk_tree_sortable_set_sort_func (GTK_TREE_SORTABLE (store), - COL_PROFILE, - profile_chooser_sort_func, - NULL, NULL); - profiles = mc_profiles_list (); for (l = profiles; l; l = l->next) { McProfile *profile; @@ -160,15 +152,26 @@ empathy_profile_chooser_new (void) } g_object_unref (protocol); - gtk_list_store_append (store, &iter); - gtk_list_store_set (store, &iter, - COL_ICON, mc_profile_get_icon_name (profile), - COL_LABEL, mc_profile_get_display_name (profile), - COL_PROFILE, profile, - -1); + gtk_list_store_insert_with_values (store, &iter, 0, + COL_ICON, mc_profile_get_icon_name (profile), + COL_LABEL, mc_profile_get_display_name (profile), + COL_PROFILE, profile, + -1); + iter_set = TRUE; } - gtk_combo_box_set_active_iter (GTK_COMBO_BOX (combo_box), &iter); + /* Set the profile sort function */ + gtk_tree_sortable_set_sort_column_id (GTK_TREE_SORTABLE (store), + COL_PROFILE, + GTK_SORT_ASCENDING); + gtk_tree_sortable_set_sort_func (GTK_TREE_SORTABLE (store), + COL_PROFILE, + profile_chooser_sort_func, + NULL, NULL); + + if (iter_set) { + gtk_combo_box_set_active_iter (GTK_COMBO_BOX (combo_box), &iter); + } mc_profiles_free_list (profiles); g_object_unref (store); -- cgit v1.2.3