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

diff --git a/src/widgets/modest-combo-box.c b/src/widgets/modest-combo-box.c
index 6020686..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;
@@ -168,8 +168,9 @@ modest_combo_box_new (const ModestPairList *pairs, GEqualFunc id_equal_func)
 		gtk_cell_layout_clear (GTK_CELL_LAYOUT(obj));
 		
 		renderer = gtk_cell_renderer_text_new ();
+		g_object_set (G_OBJECT (renderer), "ellipsize", PANGO_ELLIPSIZE_END, NULL);
 		gtk_cell_layout_pack_start (GTK_CELL_LAYOUT(obj),
-					    renderer, FALSE);  
+					    renderer, TRUE);  
 		gtk_cell_layout_set_attributes (GTK_CELL_LAYOUT(obj),
 						renderer, "text",
 						COLUMN_DISPLAY_NAME, NULL); 
@@ -205,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);
 }
 
 
@@ -223,7 +222,7 @@ modest_combo_box_set_active_id (ModestComboBox *self, gpointer id)
 	GtkTreeModel *model;
 	GtkTreeIter iter;
 	ModestComboBoxPrivate *priv;
-	gboolean found;
+	gboolean found = FALSE;
 	
 	g_return_if_fail (self);
 
@@ -236,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;
 		}
@@ -251,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;
 }