Cppcheck report - ctk (cppcheck 2.15.0-1): modules/printbackends/papi/ctkprintbackendpapi.c

/* CTK - The GIMP Toolkit
 * ctkprintbackendpapi.c: Default implementation of CtkPrintBackend 
 * for printing to papi 
 * Copyright (C) 2003, Red Hat, Inc.
 * Copyright (C) 2009, Sun Microsystems, Inc.
 *
 * This library is free software; you can redistribute it and/or
 * modify it under the terms of the GNU Lesser General Public
 * License as published by the Free Software Foundation; either
 * version 2 of the License, or (at your option) any later version.
 *
 * This library is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
 * Lesser General Public License for more details.
 *
 * You should have received a copy of the GNU Lesser General Public
 * License along with this library. If not, see <http://www.gnu.org/licenses/>.
 */

#include <unistd.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include <stdlib.h>
#include <string.h>
#include <papi.h>

#include <config.h>
#include <errno.h>
#include <cairo.h>
#include <cairo-ps.h>

#include <glib/gi18n-lib.h>

#include "ctk.h"
#include "ctkprintbackendpapi.h"
#include "ctkprinterpapi.h"
#include "ctkprinter-private.h"

typedef struct _CtkPrintBackendPapiClass CtkPrintBackendPapiClass;

#define CTK_PRINT_BACKEND_PAPI_CLASS(klass)     (G_TYPE_CHECK_CLASS_CAST ((klass), CTK_TYPE_PRINT_BACKEND_PAPI, CtkPrintBackendPapiClass))
#define CTK_IS_PRINT_BACKEND_PAPI_CLASS(klass)  (G_TYPE_CHECK_CLASS_TYPE ((klass), CTK_TYPE_PRINT_BACKEND_PAPI))
#define CTK_PRINT_BACKEND_PAPI_GET_CLASS(obj)   (G_TYPE_INSTANCE_GET_CLASS ((obj), CTK_TYPE_PRINT_BACKEND_PAPI, CtkPrintBackendPapiClass))

#define _PAPI_MAX_CHUNK_SIZE 8192

static GType print_backend_papi_type = 0;

struct _CtkPrintBackendPapiClass
{
  CtkPrintBackendClass parent_class;
};

struct _CtkPrintBackendPapi
{
  CtkPrintBackend parent_instance;

  char *default_printer;  
};

typedef struct {
  CtkPrinter *printer;<--- struct member '_PrinterStatus::printer' is never used.
} _PrinterStatus;

static GObjectClass *backend_parent_class;

static void                 ctk_print_backend_papi_class_init      (CtkPrintBackendPapiClass *class);
static void                 ctk_print_backend_papi_init            (CtkPrintBackendPapi      *impl);
static void                 ctk_print_backend_papi_finalize        (GObject                  *object);
static void                 ctk_print_backend_papi_dispose         (GObject                  *object);
static void                 papi_request_printer_list              (CtkPrintBackend          *print_backend);
static gboolean 	    papi_get_printer_list 		   (CtkPrintBackendPapi      *papi_backend);
static void                 papi_printer_request_details           (CtkPrinter               *printer);
static CtkPrintCapabilities papi_printer_get_capabilities          (CtkPrinter               *printer);
static void                 papi_printer_get_settings_from_options (CtkPrinter               *printer,
								   CtkPrinterOptionSet       *options,
								   CtkPrintSettings          *settings);
static CtkPrinterOptionSet *papi_printer_get_options               (CtkPrinter               *printer,
								   CtkPrintSettings          *settings,
								   CtkPageSetup              *page_setup,
								   CtkPrintCapabilities       capabilities);
static void                 papi_printer_prepare_for_print         (CtkPrinter               *printer,
								   CtkPrintJob               *print_job,
								   CtkPrintSettings          *settings,
								   CtkPageSetup              *page_setup);
static cairo_surface_t *    papi_printer_create_cairo_surface      (CtkPrinter               *printer,
								   CtkPrintSettings          *settings,
								   gdouble                   width,
								   gdouble                   height,
								   GIOChannel                *cache_io);
static void                 ctk_print_backend_papi_print_stream    (CtkPrintBackend          *print_backend,
								   CtkPrintJob               *job,
								   GIOChannel                *data_io,
								   CtkPrintJobCompleteFunc   callback,
								   gpointer                  user_data,
								   GDestroyNotify            dnotify);

static gboolean             papi_display_printer_status            (gpointer user_data);
static void                 papi_display_printer_status_done       (gpointer user_data);

static void
ctk_print_backend_papi_register_type (GTypeModule *module)
{
  const GTypeInfo print_backend_papi_info =
  {
    sizeof (CtkPrintBackendPapiClass),
    NULL,		/* base_init */
    NULL,		/* base_finalize */
    (GClassInitFunc) ctk_print_backend_papi_class_init,
    NULL,		/* class_finalize */
    NULL,		/* class_data */
    sizeof (CtkPrintBackendPapi),
    0,		/* n_preallocs */
    (GInstanceInitFunc) ctk_print_backend_papi_init,
  };

  print_backend_papi_type = g_type_module_register_type (module,
                                                        CTK_TYPE_PRINT_BACKEND,
                                                        "CtkPrintBackendPapi",
                                                        &print_backend_papi_info, 0);
}

G_MODULE_EXPORT void 
pb_module_init (GTypeModule *module)
{
  ctk_print_backend_papi_register_type (module);
  ctk_printer_papi_register_type (module);
}

G_MODULE_EXPORT void 
pb_module_exit (void)
{

}
  
G_MODULE_EXPORT CtkPrintBackend * 
pb_module_create (void)
{
  return ctk_print_backend_papi_new ();
}

/*
 * CtkPrintBackendPapi
 */
GType
ctk_print_backend_papi_get_type (void)
{
  return print_backend_papi_type;
}

/**
 * ctk_print_backend_papi_new:
 *
 * Creates a new #CtkPrintBackendPapi object. #CtkPrintBackendPapi
 * implements the #CtkPrintBackend interface with direct access to
 * the filesystem using Unix/Linux API calls
 *
 * Returns: the new #CtkPrintBackendPapi object
 **/
CtkPrintBackend *
ctk_print_backend_papi_new (void)
{
  return g_object_new (CTK_TYPE_PRINT_BACKEND_PAPI, NULL);
}

static void
ctk_print_backend_papi_class_init (CtkPrintBackendPapiClass *class)
{
  GObjectClass *gobject_class = G_OBJECT_CLASS (class);
  CtkPrintBackendClass *backend_class = CTK_PRINT_BACKEND_CLASS (class);
  
  backend_parent_class = g_type_class_peek_parent (class);

  gobject_class->finalize = ctk_print_backend_papi_finalize;
  gobject_class->dispose = ctk_print_backend_papi_dispose;

  backend_class->request_printer_list = papi_request_printer_list;
  backend_class->printer_request_details = papi_printer_request_details;
  backend_class->printer_get_capabilities = papi_printer_get_capabilities;<--- You might need to cast the function pointer here
  backend_class->printer_get_options = papi_printer_get_options;
  backend_class->printer_get_settings_from_options = papi_printer_get_settings_from_options;
  backend_class->printer_prepare_for_print = papi_printer_prepare_for_print;
  backend_class->printer_create_cairo_surface = papi_printer_create_cairo_surface;
  backend_class->print_stream = ctk_print_backend_papi_print_stream;
}

static cairo_status_t
_cairo_write (void                *closure,
              const unsigned char *data,
              unsigned int         length)
{
  GIOChannel *io = (GIOChannel *)closure;
  gsize written;
  GError *error = NULL;

  CTK_NOTE (PRINTING,
            g_print ("PAPI Backend: Writting %i byte chunk to temp file\n", length));

  while (length > 0) 
    {
      g_io_channel_write_chars (io, (char *)data, length, &written, &error);

      if (error != NULL)
	{
	  CTK_NOTE (PRINTING,
                     g_print ("PAPI Backend: Error writting to temp file, %s\n", error->message));

          g_error_free (error);
	  return CAIRO_STATUS_WRITE_ERROR;
	}    

      CTK_NOTE (PRINTING,
                g_print ("PAPI Backend: Wrote %i bytes to temp file\n", written));

      data += written;
      length -= written;
    }

  return CAIRO_STATUS_SUCCESS;
}

static cairo_surface_t *
papi_printer_create_cairo_surface (CtkPrinter       *printer,
				  CtkPrintSettings *settings,
				  gdouble           width, 
				  gdouble           height,
				  GIOChannel       *cache_io)
{
  cairo_surface_t *surface;
  
  surface = cairo_ps_surface_create_for_stream (_cairo_write, cache_io, width, height);

  cairo_surface_set_fallback_resolution (surface,
                                         2.0 * ctk_print_settings_get_printer_lpi (settings),
                                         2.0 * ctk_print_settings_get_printer_lpi (settings));

  return surface;
}

typedef struct {
  CtkPrintBackend *backend;
  CtkPrintJobCompleteFunc callback;
  CtkPrintJob *job;
  gpointer user_data;
  GDestroyNotify dnotify;

  papi_service_t service;
  papi_stream_t stream;
} _PrintStreamData;

static void
papi_print_cb (CtkPrintBackendPapi *print_backend,
              GError             *error,
              gpointer            user_data)
{
  _PrintStreamData *ps = (_PrintStreamData *) user_data;

  if (ps->callback)
    ps->callback (ps->job, ps->user_data, error);

  if (ps->dnotify)
    ps->dnotify (ps->user_data);

  ctk_print_job_set_status (ps->job, 
			    error ? CTK_PRINT_STATUS_FINISHED_ABORTED 
			          : CTK_PRINT_STATUS_FINISHED);

  if (ps->job)
    g_object_unref (ps->job);
  
  g_free (ps);
}

static gboolean
papi_write (GIOChannel   *source,
           GIOCondition  con,
           gpointer      user_data)
{
  gchar buf[_PAPI_MAX_CHUNK_SIZE];
  gsize bytes_read;
  GError *error;
  GIOStatus status;
  _PrintStreamData *ps = (_PrintStreamData *) user_data;
  papi_job_t job = NULL;

  error = NULL;
  status = g_io_channel_read_chars (source,
                                    buf,
                                    _PAPI_MAX_CHUNK_SIZE,
                                    &bytes_read,
                                    &error);

  /* Keep writing to PAPI input stream while there are data */
  if (status != G_IO_STATUS_ERROR)
  {
     papiJobStreamWrite (ps->service, ps->stream, buf, bytes_read);
  }
  
  /* Finish reading input stream data. Closing the stream and handle to service */
  if (bytes_read == 0) {
     papiJobStreamClose (ps->service, ps->stream, &job);
     ps->stream = NULL;
     papiJobFree (job);
     papiServiceDestroy (ps->service);
     ps->service = NULL;
  }

  if (error != NULL || status == G_IO_STATUS_EOF)
    {
      papi_print_cb (CTK_PRINT_BACKEND_PAPI (ps->backend), 
		    error, user_data);

      if (error)
	g_error_free (error);

      if (error != NULL)
        {
          CTK_NOTE (PRINTING,
                    g_print ("PAPI Backend: %s\n", error->message));

          g_error_free (error);
        } 

      return FALSE;
    }

  CTK_NOTE (PRINTING,
            g_print ("PAPI Backend: Writting %i byte chunk to papi pipe\n", bytes_read));

  return TRUE;
}

static void
ctk_print_backend_papi_print_stream (CtkPrintBackend        *print_backend,
				    CtkPrintJob            *job,
				    GIOChannel             *data_io,
				    CtkPrintJobCompleteFunc callback,
				    gpointer                user_data,
				    GDestroyNotify          dnotify)
{
  GError *print_error = NULL;<--- Variable 'print_error' can be declared as pointer to const<--- Variable 'print_error' is assigned a value that is never used.
  CtkPrinterPapi *printer;
  _PrintStreamData *ps;
  CtkPrintSettings *settings;
  const gchar *title;
  char *prtnm = NULL;
  CtkPrintDuplex val;
  papi_status_t pstatus;
  papi_attribute_t **attrs = NULL;
  papi_job_ticket_t *ticket = NULL;
  
  printer = CTK_PRINTER_PAPI (ctk_print_job_get_printer (job));
  settings = ctk_print_job_get_settings (job);

  /* FIXME - the title should be set as the job-name */
  title = ctk_print_job_get_title (job);<--- Variable 'title' is assigned a value that is never used.

  ps = g_new0 (_PrintStreamData, 1);
  ps->callback = callback;
  ps->user_data = user_data;
  ps->dnotify = dnotify;
  ps->job = g_object_ref (job);
  ps->service = NULL;
  ps->stream = NULL;

   /* This cannot be queried yet with the current API */
  papiAttributeListAddString (&attrs, PAPI_ATTR_EXCL, "document-format", "application/postscript");
  val =  ctk_print_settings_get_duplex (settings) ;
  if (val == CTK_PRINT_DUPLEX_HORIZONTAL)
    papiAttributeListAddString (&attrs, PAPI_ATTR_EXCL, "Duplex", "DuplexNoTumble");
  else if (val == CTK_PRINT_DUPLEX_VERTICAL)
    papiAttributeListAddString (&attrs, PAPI_ATTR_EXCL, "Duplex", "DuplexTumble");

  if (job->num_copies > 1) 
    {
      papiAttributeListAddInteger (&attrs, PAPI_ATTR_EXCL, "copies", job->num_copies); 
    }

  prtnm = strdup (ctk_printer_get_name (CTK_PRINTER(printer)));

  if (papiServiceCreate (&(ps->service), prtnm, NULL, NULL, NULL,
                         PAPI_ENCRYPT_NEVER, NULL) != PAPI_OK)
    return;

  pstatus = papiJobStreamOpen (ps->service, prtnm, attrs, ticket, &(ps->stream));
  if (pstatus != PAPI_OK)
    {
      papiServiceDestroy (ps->service);
      ps->service = NULL;
      return;
    }

  /* Everything set up fine, so get ready to wait for input data stream */
  g_io_add_watch (data_io,
                  G_IO_IN | G_IO_PRI | G_IO_ERR | G_IO_HUP,
                  (GIOFunc) papi_write,
                  ps);
}


static void
_papi_set_default_printer (CtkPrintBackendPapi *backend)
{
  char *def_printer = NULL;
  char *_default_attr[] = { "printer-name", NULL };
  papi_service_t service = NULL;
  papi_printer_t default_printer = NULL;
  papi_attribute_t **attrs = NULL;

  if (papiServiceCreate (&service, NULL, NULL, NULL, NULL, PAPI_ENCRYPT_NEVER,
                          NULL) != PAPI_OK)
    return;

  if (papiPrinterQuery (service, "_default", _default_attr, NULL, 
                        &default_printer) == PAPI_OK)  
    {
      if (default_printer != NULL)  
        {
          attrs = papiPrinterGetAttributeList (default_printer);

          if (attrs != NULL)
            if (papiAttributeListGetString (attrs, NULL, "printer-name", 
                                            &def_printer) == PAPI_OK) 
	      {
	        backend->default_printer = strdup (def_printer);
	      }
        }
    }

  papiPrinterFree (default_printer);
  papiServiceDestroy (service);
}

static void
ctk_print_backend_papi_init (CtkPrintBackendPapi *backend)
{
  _papi_set_default_printer (backend);
}

static void
ctk_print_backend_papi_finalize (GObject *object)
{
  CtkPrintBackendPapi *backend_papi;

  CTK_NOTE (PRINTING,
            g_print ("PAPI Backend: finalizing PAPI backend module\n"));

  backend_papi = CTK_PRINT_BACKEND_PAPI (object);

  g_free (backend_papi->default_printer);
  backend_papi->default_printer = NULL;

  backend_parent_class->finalize (object);
}


static void
ctk_print_backend_papi_dispose (GObject *object)
{
  CtkPrintBackendPapi *backend_papi;

  CTK_NOTE (PRINTING,
            g_print ("PAPI Backend: %s\n", G_STRFUNC));

  backend_papi = CTK_PRINT_BACKEND_PAPI (object);

  backend_parent_class->dispose (object);
}

char **
get_all_list(papi_service_t svc)
{
        papi_status_t status;
        papi_printer_t printer = NULL;
        char *attr[] = { "member-names", NULL };
        char **names = NULL;

        status = papiPrinterQuery(svc, "_all", attr, NULL, &printer);
        if ((status == PAPI_OK) && (printer != NULL)) {
                papi_attribute_t **attributes =
                                        papiPrinterGetAttributeList(printer);
                if (attributes != NULL) {
                        void *iter = NULL;
                        char *member = NULL;

                        for (status = papiAttributeListGetString(attributes,
                                                &iter, "member-names", &member);
                                status == PAPI_OK;
                                status = papiAttributeListGetString(attributes,
                                                &iter, NULL, &member))
                                        list_append(&names, strdup(member));
                }
                papiPrinterFree(printer);
        }

        return (names);
}

static char **
get_printers_list(papi_service_t svc)
{
	papi_status_t status;
    	papi_printer_t *printers = NULL;
    	char *keys[] = { "printer-name", "printer-uri-supported", NULL };
    	char **names = NULL;

	status = papiPrintersList(svc, keys, NULL, &printers);
	if ((status == PAPI_OK) && (printers != NULL)) {
		int i;

		for (i = 0; printers[i] != NULL; i++) {
			papi_attribute_t **attributes =
				papiPrinterGetAttributeList(printers[i]);
    			char *name = NULL;
			
			(void) papiAttributeListGetString(attributes, NULL,
							"printer-name", &name);
			if ((name != NULL) && (strcmp(name, "_default") != 0))
    				list_append(&names, strdup(name));
    		}
    		papiPrinterListFree(printers);
    	}
    
    	return (names);
}

static void
papi_request_printer_list (CtkPrintBackend *backend)
{
  CtkPrintBackendPapi *papi_backend;

  papi_backend = CTK_PRINT_BACKEND_PAPI (backend);

  /* Get the list of printers using papi API */
  papi_get_printer_list (papi_backend); 
}

static gboolean
papi_get_printer_list (CtkPrintBackendPapi *papi_backend)
{
  int i;
  papi_status_t status;
  papi_service_t service = NULL;
  char **printers = NULL;
  CtkPrinterPapi *papi_printer;
  CtkPrintBackend *backend = CTK_PRINT_BACKEND (papi_backend);
  
  if ((status = papiServiceCreate (&service, NULL, NULL, NULL, NULL,
		PAPI_ENCRYPT_NEVER, NULL)) != PAPI_OK)
    return FALSE; 

  if ((printers = get_all_list (service)) == NULL) 
    {
      printers = get_printers_list (service);
    }

  if (printers == NULL) 
    {
      papiServiceDestroy (service);
      return FALSE;
    }

  for (i = 0; printers[i] != NULL; i++) 
    {
      CtkPrinter *printer;

          printer = ctk_print_backend_find_printer (backend, printers[i]);

	  if (!printer) 
	    {
              /* skip null printer name just in case */
              if (printers[i] == NULL)
                continue;

	      /* skip the alias _default and _all printers */
      	      if (strcmp(printers[i], "_default")==0 || strcmp(printers[i], "_all")==0)
	        continue;	

	      papi_printer = ctk_printer_papi_new (printers[i], backend);
	      printer = CTK_PRINTER (papi_printer);

	      /* Only marked default printer to not have details so that
		 the request_details method will be called  at start up
	       */

	      if (papi_backend->default_printer != NULL)
	        if (strcmp (printers[i], papi_backend->default_printer)==0)
		  {
		    ctk_printer_set_is_default (printer, TRUE);
	  	  }	

              ctk_printer_set_icon_name (printer, "printer");
	      ctk_print_backend_add_printer (backend, printer);
              ctk_printer_set_is_active (printer, TRUE);

  	      /* ctk_printer_set_has_details (printer, TRUE); */
	    }
    	  else 
            g_object_ref (printer);

      if (!ctk_printer_is_active (printer))
        {
          ctk_printer_set_is_active (printer, TRUE);
          ctk_printer_set_is_new (printer, TRUE);
        }

      if (ctk_printer_is_new (printer))
        {
          g_signal_emit_by_name (backend, "printer-added", printer);
          ctk_printer_set_is_new (printer, FALSE);
        }

      g_object_unref (printer);
    }

  free (printers);
  papiServiceDestroy (service);

  /* To set that the list of printers added is complete */
  ctk_print_backend_set_list_done (backend); 

  return TRUE;
}

static void
update_printer_status (CtkPrinter *printer)
{
  CtkPrintBackend *backend;
  CtkPrinterPapi *papi_printer;

  backend = ctk_printer_get_backend (printer);
  papi_printer = CTK_PRINTER_PAPI (printer);

  g_signal_emit_by_name (CTK_PRINT_BACKEND (backend),
                         "printer-status-changed", printer);

}


static CtkPrinterOptionSet *
papi_printer_get_options (CtkPrinter           *printer,
			  CtkPrintSettings     *settings,
			  CtkPageSetup         *page_setup,
			  CtkPrintCapabilities  capabilities)
{
  CtkPrinterOptionSet *set;
  CtkPrinterOption *option;
  char *print_at[] = { "now", "on-hold" };
  char *n_up[] = {"1"};

  /* Update the printer status before the printer options are displayed */
  update_printer_status (printer); 

  set = ctk_printer_option_set_new ();

  /* non-ppd related settings */

  /* This maps to number-up-supported in PAPI. FIXME 
   * number-up-default is the default value. 
   * number-up-supported is the list of number of able to print per page 
   */
  option = ctk_printer_option_new ("ctk-n-up", "Pages Per Sheet", CTK_PRINTER_OPTION_TYPE_PICKONE);
  ctk_printer_option_choices_from_array (option, G_N_ELEMENTS (n_up),
					 n_up, n_up);
  ctk_printer_option_set (option, "1");
  ctk_printer_option_set_add (set, option);
  g_object_unref (option);

  /* This maps to job-priority-supported and job-priority-default in PAPI - FIXME*/
  
  /* This relates to job-sheets-supported in PAPI  FIXME*/
  
  /* This relates to job-hold-until-supported in PAPI */
  option = ctk_printer_option_new ("ctk-print-time", "Print at", CTK_PRINTER_OPTION_TYPE_PICKONE);
  ctk_printer_option_choices_from_array (option, G_N_ELEMENTS (print_at),
					 print_at, print_at);
  ctk_printer_option_set (option, "now");
  ctk_printer_option_set_add (set, option);
  g_object_unref (option);
  
  return set;
}

static void
papi_printer_get_settings_from_options (CtkPrinter          *printer,
				       CtkPrinterOptionSet *options,
				       CtkPrintSettings    *settings)
{
  CtkPrinterOption *option;

  option = ctk_printer_option_set_lookup (options, "ctk-n-up");
  if (option)
     ctk_print_settings_set (settings, CTK_PRINT_SETTINGS_NUMBER_UP, option->value);

}

static void
papi_printer_prepare_for_print (CtkPrinter       *printer,
			       CtkPrintJob      *print_job,
			       CtkPrintSettings *settings,
			       CtkPageSetup     *page_setup)
{
  CtkPageSet page_set;
  double scale;
  CtkPaperSize *papersize = NULL;
  char *ppd_paper_name;<--- Variable 'ppd_paper_name' can be declared as pointer to const

  print_job->print_pages = ctk_print_settings_get_print_pages (settings);
  print_job->page_ranges = NULL;
  print_job->num_page_ranges = 0;
  
  if (print_job->print_pages == CTK_PRINT_PAGES_RANGES)
    print_job->page_ranges =
      ctk_print_settings_get_page_ranges (settings,
					  &print_job->num_page_ranges);
  
  print_job->collate = ctk_print_settings_get_collate (settings);
  print_job->reverse = ctk_print_settings_get_reverse (settings);
  print_job->num_copies = ctk_print_settings_get_n_copies (settings);
 
  scale = ctk_print_settings_get_scale (settings);
  if (scale != 100.0)
    print_job->scale = scale/100.0;

  papersize = ctk_page_setup_get_paper_size (page_setup);
  ppd_paper_name = ctk_paper_size_get_ppd_name (papersize);<--- Variable 'ppd_paper_name' is assigned a value that is never used.

  page_set = ctk_print_settings_get_page_set (settings);
  if (page_set == CTK_PAGE_SET_EVEN)
    print_job->page_set = CTK_PAGE_SET_EVEN;
  else if (page_set == CTK_PAGE_SET_ODD)
    print_job->page_set = CTK_PAGE_SET_ODD;
  else
    print_job->page_set = CTK_PAGE_SET_ALL;

  print_job->rotate_to_orientation = TRUE;

}

gboolean
is_local_printer (gchar *printer_uri)<--- Parameter 'printer_uri' can be declared as pointer to const
{
  if (strncmp (printer_uri, "lpsched:", 8) == 0)
    return TRUE;
  else
    return FALSE;
}

static void
papi_display_printer_status_done (gpointer user_data)
{
  CtkPrinter *printer = (CtkPrinter *) user_data;
  CtkPrinterPapi *papi_printer;

  g_signal_emit_by_name (printer, "details-acquired", TRUE); 
  papi_printer = CTK_PRINTER_PAPI (printer);
  return;
}

#define IDLE 3
#define PROCESSING 4
#define STOPPED 5
static gboolean
papi_display_printer_status (gpointer user_data)
{
  CtkPrinter *printer = (CtkPrinter *) user_data;
  CtkPrinterPapi *papi_printer;
  gchar *loc;
  int state;
  papi_service_t service;
  papi_attribute_t **attrs = NULL;
  papi_printer_t current_printer = NULL;
  static int count = 0;<--- Variable 'count' is assigned a value that is never used.

  papi_printer = CTK_PRINTER_PAPI (printer);
  if (papiServiceCreate (&service, NULL, NULL, NULL, NULL, PAPI_ENCRYPT_NEVER,
                          NULL) != PAPI_OK)
    return G_SOURCE_REMOVE;

  if (papiPrinterQuery (service, papi_printer->printer_name, NULL, NULL,
                        &current_printer) != PAPI_OK) 
    {
       /* SUN_BRANDING */
       ctk_printer_set_state_message (printer, _("printer offline"));
    }

  if (current_printer != NULL)
    {
        attrs = papiPrinterGetAttributeList (current_printer);
    }

  if (papiAttributeListGetString (attrs, NULL, "printer-info", &loc) == PAPI_OK)
    {
        ctk_printer_set_location (printer, loc);
    }

  if (papiAttributeListGetInteger (attrs, NULL, "printer-state", &state) == PAPI_OK)
    {
      switch (state) 
        {
	  /* SUN_BRANDING */
	  case IDLE: ctk_printer_set_state_message (printer, _("ready to print"));
		     break;
	  /* SUN_BRANDING */
	  case PROCESSING: ctk_printer_set_state_message (printer, _("processing job"));
		           break;

	  /* SUN_BRANDING */
	  case STOPPED: ctk_printer_set_state_message (printer, _("paused"));
		        break;
	  /* SUN_BRANDING */
	  default: ctk_printer_set_state_message (printer, _("unknown"));
		   break;
        }
    }

  papiPrinterFree (current_printer);
  papiServiceDestroy (service);
  ctk_printer_set_has_details (printer, TRUE);

  return G_SOURCE_REMOVE;
}

static void  
papi_printer_request_details (CtkPrinter *printer)
{
  g_idle_add_full (G_PRIORITY_DEFAULT_IDLE, papi_display_printer_status, printer, papi_display_printer_status_done); 
}


static CtkPrintCapabilities
papi_printer_get_capabilities (CtkPrinter *printer)<--- Parameter 'printer' can be declared as pointer to const
{
  return CTK_PRINT_CAPABILITY_COPIES | CTK_PRINT_CAPABILITY_PAGE_SET ; 
}