Code cleanups in HildonTouchSelector

2009-04-22  Alberto Garcia  <agarcia@igalia.com>

	* src/hildon-touch-selector.c
	(hildon_touch_selector_append_text_column):
	Code cleanups

diff --git a/ChangeLog b/ChangeLog
index 339d324..1718fc0 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2009-04-22  Alberto Garcia  <agarcia@igalia.com>
+
+       * src/hildon-touch-selector.c
+       (hildon_touch_selector_append_text_column):
+       Code cleanups
+
 2009-04-22  Claudio Saavedra  <csaavedra@igalia.com>
 
        * src/hildon-number-editor.c (hildon_number_editor_real_set_value):

diff --git a/src/hildon-touch-selector.c b/src/hildon-touch-selector.c
index ae7d70b..26b7e13 100644 (file)
--- a/src/hildon-touch-selector.c
+++ b/src/hildon-touch-selector.c
@@ -1196,26 +1196,20 @@ HildonTouchSelectorColumn*
 hildon_touch_selector_append_text_column (HildonTouchSelector * selector,
                                           GtkTreeModel * model, gboolean center)
 {
-  GtkCellRenderer *renderer = NULL;
-  GValue val = { 0, };
+  gfloat xalign = center ? 0.5 : 0.0;
+  GtkCellRenderer *renderer;
 
   g_return_val_if_fail (HILDON_IS_TOUCH_SELECTOR (selector), NULL);
   g_return_val_if_fail (GTK_IS_TREE_MODEL (model), NULL);
 
-  if (model != NULL) {
-    renderer = gtk_cell_renderer_text_new ();
+  renderer = gtk_cell_renderer_text_new ();
 
-    if (center) {
-      g_value_init (&val, G_TYPE_FLOAT);
-      g_value_set_float (&val, 0.5);
-      g_object_set_property (G_OBJECT (renderer), "xalign", &val);
-    }
+  g_object_set (renderer,
+                "xalign", xalign,
+                NULL);
 
-    return hildon_touch_selector_append_column (selector, model, renderer,
-                                                "text", 0, NULL);
-  } else {
-    return NULL;
-  }
+  return hildon_touch_selector_append_column (selector, model, renderer,
+                                              "text", 0, NULL);
 }
 
 /**