[Window] Fix shadowed variable "index"
authorIra W. Snyder <devel@irasnyder.com>
Tue, 29 Jan 2008 02:46:58 +0000 (18:46 -0800)
committerIra W. Snyder <devel@irasnyder.com>
Tue, 29 Jan 2008 02:46:58 +0000 (18:46 -0800)
The string.h header declares a global variable "index". The TildaWindow
code also used a variable named index, so the global variable was
shadowed by the local variable. While this isn't a problem, I prefer to
make code -Wshadow safe.

Also, I removed a comment left over from debugging in TildaController.

tilda-controller.c
tilda-window.c

index a3d6f67..7361ff8 100644 (file)
@@ -240,7 +240,6 @@ tilda_controller_constructor (GType                  type,
        obj = parent_class->constructor (type,
                                                                         n_construct_properties,
                                                                         construct_properties);
-       g_print ("%s all set up\n", __func__);
 
        /* The object is ready, and all constructor-time properties have been set.
         * Have fun! */
index 822a987..6a5b84e 100644 (file)
@@ -91,6 +91,8 @@ tilda_window_add_terminal (TildaWindow *self)
 
        gint number;
        TildaTerminal *tt;
+       GtkWidget *label;
+       gint notebook_index;
 
        number = tilda_window_find_next_free_terminal_number (self);
        tt = g_object_new (TILDA_TYPE_TERMINAL,
@@ -99,10 +101,10 @@ tilda_window_add_terminal (TildaWindow *self)
                                           NULL);
        g_ptr_array_add (self->terms, tt);
 
-       GtkWidget *label = gtk_label_new ("Tilda");
-       gint index = gtk_notebook_prepend_page (GTK_NOTEBOOK(self->notebook), tt->hbox, label);
+       label = gtk_label_new ("Tilda");
+       notebook_index = gtk_notebook_prepend_page (GTK_NOTEBOOK(self->notebook), tt->hbox, label);
        gtk_notebook_set_tab_label_packing (GTK_NOTEBOOK(self->notebook), tt->hbox, TRUE, TRUE, GTK_PACK_END);
-       gtk_notebook_set_current_page (GTK_NOTEBOOK(self->notebook), index);
+       gtk_notebook_set_current_page (GTK_NOTEBOOK(self->notebook), notebook_index);
 
        if (gtk_notebook_get_n_pages (GTK_NOTEBOOK(self->notebook)) > 1)
                gtk_notebook_set_show_tabs (GTK_NOTEBOOK(self->notebook), TRUE);