Instead of having a single enumerated type for status return values from

cf_ functions, have separate ones for different classes of routines, and
use gboolean when the return value is just "success" or "failure" - that
way you don't get compiler warnings if a case statement isn't handling a
particular status value if the routine in question won't return it.

svn path=/trunk/; revision=13293

1 files changed, 9 insertions, 9 deletions

diff --git a/gtk/print_dlg.c b/gtk/print_dlg.c
index 5163db6fbc..2856bf8cc8 100644
--- a/gtk/print_dlg.c
+++ b/gtk/print_dlg.c
@@ -760,16 +760,16 @@ print_cmd_toggle_detail(GtkWidget *widget _U_, gpointer data)
 static void
 print_ok_cb(GtkWidget *ok_bt, gpointer parent_w)
 {
-  GtkWidget     *button;
-  print_args_t  *args;
-  const gchar   *g_dest;
-  gchar         *f_name;
-  gchar         *dirname;
-  gboolean      export_as_pdml = FALSE, export_as_psml = FALSE;
+  GtkWidget         *button;
+  print_args_t      *args;
+  const gchar       *g_dest;
+  gchar             *f_name;
+  gchar             *dirname;
+  gboolean          export_as_pdml = FALSE, export_as_psml = FALSE;
 #ifdef _WIN32
-  gboolean win_printer = FALSE;
+  gboolean          win_printer = FALSE;
 #endif
-  cf_status_t   status;
+  cf_print_status_t status;
 
   args = (print_args_t *)OBJECT_GET_DATA(ok_bt, PRINT_ARGS_KEY);
 
@@ -903,7 +903,7 @@ print_ok_cb(GtkWidget *ok_bt, gpointer parent_w)
   }
   switch (status) {
 
-  case CF_OK:
+  case CF_PRINT_OK:
     break;
 
   case CF_PRINT_OPEN_ERROR: