2009-01-14 Claudio Saavedra <csaavedra@igalia.com>

* src/hildon-picker-dialog.c: (_save_current_selection): Do not query
	for the number of columns in the HildonTouchSelector more than once.

2 files changed, 8 insertions, 2 deletions

diff --git a/ChangeLog b/ChangeLog
index 949d173..f49ed96 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2009-01-14  Claudio Saavedra  <csaavedra@igalia.com>
+
+	* src/hildon-picker-dialog.c: (_save_current_selection): Do not query
+	for the number of columns in the HildonTouchSelector more than once.
+
 2009-01-14  Alberto Garcia  <agarcia@igalia.com>
 
 	* src/hildon-touch-selector.c: Minor documentation fixes.
diff --git a/src/hildon-picker-dialog.c b/src/hildon-picker-dialog.c
index e701440..e803593 100644
--- a/src/hildon-picker-dialog.c
+++ b/src/hildon-picker-dialog.c
@@ -366,13 +366,14 @@ static void
 _save_current_selection (HildonPickerDialog *dialog)
 {
   HildonTouchSelector *selector;
-  gint i;
+  gint i, columns;
 
   selector = HILDON_TOUCH_SELECTOR (dialog->priv->selector);
 
   _clean_current_selection (dialog);
 
-  for (i = 0; i  < hildon_touch_selector_get_num_columns (selector); i++) {
+  columns = hildon_touch_selector_get_num_columns (selector);
+  for (i = 0; i  < columns; i++) {
     dialog->priv->current_selection
       = g_slist_append (dialog->priv->current_selection,
                         hildon_touch_selector_get_selected_rows (selector, i));