X-Git-Url: http://git.maemo.org/git/?p=modest;a=blobdiff_plain;f=src%2Fwidgets%2Fmodest-combo-box.c;h=8956df873694b7cc0e4ebff278d393136c0a74c4;hp=e44aef1d264ea92f56eabb4b93f8cc0de33d76b4;hb=46a234c81bceecac16ac22ae3f281b79247d46f7;hpb=326ddf804e1bc2542650028660f4eb5cc82af96a

diff --git a/src/widgets/modest-combo-box.c b/src/widgets/modest-combo-box.c
index e44aef1..8956df8 100644
--- a/src/widgets/modest-combo-box.c
+++ b/src/widgets/modest-combo-box.c
@@ -125,11 +125,11 @@ modest_combo_box_finalize (GObject *obj)
 }
 
 static GtkTreeModel*
-get_model (const ModestPairList *pairs)
+get_model (ModestPairList *pairs)
 {
 	GtkTreeIter iter;
 	GtkListStore *store;
-	const GSList *cursor;
+	GSList *cursor;
 	
 	store = gtk_list_store_new (2,
 				    G_TYPE_POINTER, /* the id */
@@ -149,7 +149,7 @@ get_model (const ModestPairList *pairs)
 
 
 GtkWidget*
-modest_combo_box_new (const ModestPairList *pairs, GEqualFunc id_equal_func)
+modest_combo_box_new (ModestPairList *pairs, GEqualFunc id_equal_func)
 {
 	GtkTreeModel *model;
 	GtkCellRenderer *renderer;
@@ -206,15 +206,13 @@ gpointer
 modest_combo_box_get_active_id (ModestComboBox *self)
 {
 	GValue val = {0,};
-	gpointer retval;
 
 	g_return_val_if_fail (self, NULL);
 	
 	/* Do not unset the GValue */
 	get_active (self, &val, COLUMN_ID);
-	retval = g_value_peek_pointer (&val);
 
-	return retval;
+	return g_value_get_pointer (&val);
 }
 
 
@@ -237,7 +235,7 @@ modest_combo_box_set_active_id (ModestComboBox *self, gpointer id)
 	do {
 		gpointer row_id;
 		gtk_tree_model_get (model, &iter, COLUMN_ID, &row_id, -1);
-		if ((priv->id_equal_func)(id, row_id) == 0) {
+		if ((priv->id_equal_func)(id, row_id)) {
 			gtk_combo_box_set_active_iter (GTK_COMBO_BOX(self), &iter);
 			found = TRUE;
 		}
@@ -252,14 +250,14 @@ modest_combo_box_set_active_id (ModestComboBox *self, gpointer id)
 const gchar*
 modest_combo_box_get_active_display_name (ModestComboBox *self)
 {
-	GValue val = {0,};
-	gpointer retval;
+	const GValue val = {0,};
+	const gchar *retval;
 
 	g_return_val_if_fail (self, NULL);
 
 	/* Do not unset the GValue */
-	get_active (self, &val, COLUMN_DISPLAY_NAME);
-	retval = g_value_peek_pointer (&val);
+	get_active (self, (GValue *)&val, COLUMN_DISPLAY_NAME);
+	retval = g_value_get_string (&val);
 
-	return (gchar*) retval;
+	return retval;
 }