MEANS: Fix potential errors in gui syntax generation
authorJohn Darrington <john@darrington.wattle.id.au>
Sat, 5 Oct 2013 10:30:12 +0000 (12:30 +0200)
committerJohn Darrington <john@darrington.wattle.id.au>
Sun, 6 Oct 2013 14:32:45 +0000 (16:32 +0200)
src/ui/gui/psppire-dialog-action-means.c

index 64d0f32876502f342b6ff10c43f23993dbf32641..6ebda5055180cc04291a72fff93d40d441ac891d 100644 (file)
@@ -51,8 +51,10 @@ generate_syntax (PsppireDialogAction *act)
       GtkTreeIter iter;
       PsppireVarView *vv = PSPPIRE_VAR_VIEW (layer->var_view);
       psppire_var_view_set_current_model (vv, l);
-      g_string_append (string, "\n\tBY");
-      for (ok = psppire_var_view_get_iter_first (vv, &iter);
+      ok = psppire_var_view_get_iter_first (vv, &iter);
+      if (ok)
+       g_string_append (string, "\n\tBY");
+      for (;
           ok;
           ok = psppire_var_view_get_iter_next (vv, &iter))
          {
@@ -62,7 +64,7 @@ generate_syntax (PsppireDialogAction *act)
          }
     }
 
-  g_string_append (string, ".");
+  g_string_append (string, ".\n");
   text = string->str;
 
   g_string_free (string, FALSE);