Dont show range if it is null
authorJohn Darrington <john@darrington.wattle.id.au>
Tue, 29 Jan 2013 19:04:59 +0000 (20:04 +0100)
committerJohn Darrington <john@darrington.wattle.id.au>
Sat, 16 Feb 2013 14:04:02 +0000 (15:04 +0100)
src/data/spreadsheet-reader.c
src/ui/gui/page-sheet-spec.c

index e34fcb38faa8045e216ffc99bc0c2e179b437f8f..2b4178835a2dae2af78e1c9d4deb34bfeb42bd01 100644 (file)
@@ -126,9 +126,18 @@ int_to_ps26 (int i)
 char *
 create_cell_ref (int col0, int row0, int coli, int rowi)
 {
-  char *cs0 =  int_to_ps26 (col0);
-  char *csi =  int_to_ps26 (coli);
-  char *s =  c_xasprintf ("%s%d:%s%ld",
+  char *cs0 ;
+  char *csi ;
+  char *s ;
+
+  if ( col0 < 0) return NULL;
+  if ( rowi < 0) return NULL;
+  if ( coli < 0) return NULL;
+  if ( row0 < 0) return NULL;
+
+  cs0 =  int_to_ps26 (col0);
+  csi =  int_to_ps26 (coli);
+  s =  c_xasprintf ("%s%d:%s%ld",
                         cs0, row0 + 1,
                         csi, rowi + 1);
   free (cs0);
index 501dc1b86b8e8629bad9a71ad884bf7810abdb2b..fe8f90f3b7155e4c8a888c30697e3909f3aa9036 100644 (file)
@@ -91,7 +91,7 @@ on_sheet_combo_changed (GtkComboBox *cb, struct import_assistant *ia)
 
   gtk_combo_box_get_active_iter (cb, &iter);
   gtk_tree_model_get (model, &iter, PSPPIRE_SPREADSHEET_MODEL_COL_RANGE, &range, -1);
-  gtk_entry_set_text (GTK_ENTRY (range_entry), range);
+  gtk_entry_set_text (GTK_ENTRY (range_entry), range ?  range : "");
   g_free (range);
 }