Author: pollux Date: 2006-10-20 20:07:42 +0000 (Fri, 20 Oct 2006) New Revision: 23487
Modified: xfprint/trunk/libxfprint/printing-system.c xfprint/trunk/xfprint/main.c xfprint/trunk/xfprint/print_dialog.c Log: fix warnings Modified: xfprint/trunk/libxfprint/printing-system.c =================================================================== --- xfprint/trunk/libxfprint/printing-system.c 2006-10-20 13:44:54 UTC (rev 23486) +++ xfprint/trunk/libxfprint/printing-system.c 2006-10-20 20:07:42 UTC (rev 23487) @@ -273,7 +273,8 @@ GList * printing_system_get_printers (PrintingSystem * ps) { - g_return_val_if_fail (ps, NULL); + if (ps == NULL) + return NULL; return PRINTING_SYSTEM_GET_PRIVATE (ps)->ps_get_printers (); } @@ -281,7 +282,8 @@ Printer * printing_system_get_default_printer (PrintingSystem * ps) { - g_return_val_if_fail (ps, NULL); + if (ps == NULL) + return NULL; return PRINTING_SYSTEM_GET_PRIVATE (ps)->ps_get_default_printer (); } @@ -289,7 +291,8 @@ gint printing_system_get_printer_state (PrintingSystem * ps, const gchar * printer) { - g_return_val_if_fail (ps, PRINTER_STATE_UNKNOWN); + if (ps == NULL) + return PRINTER_STATE_UNKNOWN; return PRINTING_SYSTEM_GET_PRIVATE (ps)->ps_get_printer_state (printer); } @@ -297,7 +300,8 @@ gint printing_system_get_jobs_count (PrintingSystem * ps, const gchar * printer) { - g_return_val_if_fail (ps, 0); + if (ps == NULL) + return 0; return PRINTING_SYSTEM_GET_PRIVATE (ps)->ps_get_printer_jobs_count (printer); } @@ -305,7 +309,8 @@ gboolean printing_system_remove_job (PrintingSystem * ps, const gchar * printer, gint id) { - g_return_val_if_fail (ps, FALSE); + if (ps == NULL) + return FALSE; return PRINTING_SYSTEM_GET_PRIVATE (ps)->ps_remove_job (printer, id); } @@ -313,7 +318,8 @@ GList * printing_system_get_jobs (PrintingSystem * ps, const gchar * printer) { - g_return_val_if_fail (ps, NULL); + if (ps == NULL) + return NULL; return PRINTING_SYSTEM_GET_PRIVATE (ps)->ps_get_jobs (printer); } @@ -321,7 +327,8 @@ gboolean printing_system_print_file (PrintingSystem * ps, const gchar * printer, const gchar * original_name, const gchar * file, gboolean remove_file) { - g_return_val_if_fail (ps, FALSE); + if (ps == NULL) + return FALSE; return PRINTING_SYSTEM_GET_PRIVATE (ps)->ps_print_file (printer, original_name, file, remove_file); } @@ -329,7 +336,8 @@ void printing_system_customize_printer_list_window (PrintingSystem * ps, PrinterListWindow * win) { - g_return_if_fail (ps); + if (ps == NULL) + return; PRINTING_SYSTEM_GET_PRIVATE (ps)->ps_customize_printer_list_window (win); } @@ -337,7 +345,8 @@ void printing_system_customize_printer_queue_window (PrintingSystem * ps, PrinterQueueWindow * win) { - g_return_if_fail (ps); + if (ps == NULL) + return; PRINTING_SYSTEM_GET_PRIVATE (ps)->ps_customize_printer_queue_window (win); } Modified: xfprint/trunk/xfprint/main.c =================================================================== --- xfprint/trunk/xfprint/main.c 2006-10-20 13:44:54 UTC (rev 23486) +++ xfprint/trunk/xfprint/main.c 2006-10-20 20:07:42 UTC (rev 23487) @@ -52,20 +52,11 @@ Display *dpy; int screen; PrintingSystem *ps = NULL; + + g_log_set_always_fatal (G_LOG_LEVEL_CRITICAL); -#if 0 -#ifdef ENABLE_NLS /* This is required for UTF-8 at least - Please don't remove it */ - bindtextdomain (GETTEXT_PACKAGE, PACKAGE_LOCALE_DIR); -#ifdef HAVE_BIND_TEXTDOMAIN_CODESET - bind_textdomain_codeset (GETTEXT_PACKAGE, "UTF-8"); -#endif - textdomain (GETTEXT_PACKAGE); -#endif -#else - /* This is required for UTF-8 at least - Please don't remove it */ xfce_textdomain (GETTEXT_PACKAGE, PACKAGE_LOCALE_DIR, "UTF-8"); -#endif gtk_set_locale (); gtk_init (&argc, &argv); @@ -89,11 +80,17 @@ g_warning ("Unable to load printing system module %s", setting->data.v_string); } mcs_setting_free (setting); + } else { + g_warning ("%s: XfPrint/system is not set", PACKAGE); } mcs_client_destroy (client); + } else { + g_warning ("%s: no mcs client found", PACKAGE); } + if (ps == NULL) + g_warning ("%s: No printing system loaded", PACKAGE); if (argc == 1) { struct timeval tv; Modified: xfprint/trunk/xfprint/print_dialog.c =================================================================== --- xfprint/trunk/xfprint/print_dialog.c 2006-10-20 13:44:54 UTC (rev 23486) +++ xfprint/trunk/xfprint/print_dialog.c 2006-10-20 20:07:42 UTC (rev 23487) @@ -413,10 +413,12 @@ printer = g_list_next (printer); } + + if (G_LIKELY (G_IS_OBJECT (icon_printer))) + g_object_unref (icon_printer); + if (G_LIKELY (G_IS_OBJECT (icon_class))) + g_object_unref (icon_class); - g_object_unref (icon_printer); - g_object_unref (icon_class); - printers_free (printers); } _______________________________________________ Xfce4-commits mailing list Xfce4-commits@xfce.org http://foo-projects.org/mailman/listinfo/xfce4-commits