[gtk+/gtk-2-24] Move some variable declarations into the scopes in which they're used.

[John Ralls <jralls src gnome org>]

commit 7d2725e40033f2a379fc23200711d4d1f7a1b00e
Author: John Ralls <jralls ceridwen us>
Date:   Sat Feb 25 17:52:46 2012 -0800

    Move some variable declarations into the scopes in which they're used.

 modules/printbackends/cups/gtkprintbackendcups.c |   39 +++++++++++----------
 1 files changed, 20 insertions(+), 19 deletions(-)
---
diff --git a/modules/printbackends/cups/gtkprintbackendcups.c b/modules/printbackends/cups/gtkprintbackendcups.c
index e2d9192..a51f681 100644
--- a/modules/printbackends/cups/gtkprintbackendcups.c
+++ b/modules/printbackends/cups/gtkprintbackendcups.c
@@ -2025,12 +2025,8 @@ cups_request_printer_list_cb (GtkPrintBackendCups *cups_backend,
       gboolean status_changed = FALSE;
       GList *node;
       gint i;
-      gchar *reason_msg_desc = NULL;
-      gchar *tmp_msg = NULL;
-      gchar *tmp_msg2 = NULL;
-      gboolean found = FALSE;
       PrinterSetupInfo *info = g_slice_new0 (PrinterSetupInfo);
-      
+
       /* Skip leading attributes until we hit a printer...
        */
       while (attr != NULL && ippGetGroupTag (attr) != IPP_TAG_PRINTER)
@@ -2126,23 +2122,30 @@ cups_request_printer_list_cb (GtkPrintBackendCups *cups_backend,
 
       if (info->state_msg != NULL && strlen (info->state_msg) == 0)
         {
-          if (info->is_paused && !info->is_accepting_jobs)
-		  /* Translators: this is a printer status. */
+	  gchar *tmp_msg2 = NULL;
+	  if (info->is_paused && !info->is_accepting_jobs)
+	    /* Translators: this is a printer status. */
             tmp_msg2 = g_strdup ( N_("Paused ; Rejecting Jobs"));
           if (info->is_paused && info->is_accepting_jobs)
-		  /* Translators: this is a printer status. */
+	    /* Translators: this is a printer status. */
             tmp_msg2 = g_strdup ( N_("Paused"));
           if (!info->is_paused && !info->is_accepting_jobs)
-		  /* Translators: this is a printer status. */
+	    /* Translators: this is a printer status. */
             tmp_msg2 = g_strdup ( N_("Rejecting Jobs"));
 
           if (tmp_msg2 != NULL)
-            info->state_msg = tmp_msg2;
-        }
+	    {
+	      info->state_msg = tmp_msg2;
+	      g_free (tmp_msg2);
+	    }
+	}
 
       /* Set description of the reason and combine it with printer-state-message. */
       if ( (info->reason_msg != NULL))
         {
+	  gchar *reason_msg_desc = NULL;
+	  gboolean found = FALSE;
+
           for (i = 0; i < G_N_ELEMENTS (printer_messages); i++)
             {
               if (strncmp (info->reason_msg, printer_messages[i],
@@ -2164,23 +2167,21 @@ cups_request_printer_list_cb (GtkPrintBackendCups *cups_backend,
                 info->state_msg = reason_msg_desc;
               else
                 {
-                  tmp_msg = g_strjoin (" ; ", info->state_msg, reason_msg_desc, NULL);
+		  gchar *tmp_msg = NULL;
+		  tmp_msg = g_strjoin (" ; ", info->state_msg,
+				       reason_msg_desc, NULL);
                   info->state_msg = tmp_msg;
+		  g_free (tmp_msg);
                 }
             }
+	  if (reason_msg_desc != NULL)
+	    g_free (reason_msg_desc);
         }
 
       status_changed |= gtk_printer_set_state_message (printer, info->state_msg);
       status_changed |= gtk_printer_set_is_accepting_jobs (printer, info->is_accepting_jobs);
 
-      if (tmp_msg != NULL)
-        g_free (tmp_msg);
-
-      if (tmp_msg2 != NULL)
-        g_free (tmp_msg2);
 
-      if (reason_msg_desc != NULL)
-        g_free (reason_msg_desc);
 
       /* Set printer icon according to importance
          (none, report, warning, error - report is omitted). */