Remove superfluous casts and fix whitespace sillyness.
[pspp-builds.git] / lib / gtksheet / gtksheet.c
index bdcef65d8f308ed293fe9b21e63f0d0a989cfea8..bc4c9e359bda9297a8413b144960338edd89cf96 100644 (file)
@@ -1,5 +1,11 @@
-/* This version of GtkSheet has been *heavily* modified, for the specific
-   requirements of PSPPIRE. */
+/*
+ * Copyright (C) 2006, 2008 Free Software Foundation
+ *
+ * This version of GtkSheet has been *heavily* modified, for the specific
+ * requirements of PSPPIRE.  The changes are copyright by the
+ * Free Software Foundation.  The copyright notice for the original work is
+ * below.
+*/
 
 /* GtkSheet widget for Gtk+.
  * Copyright (C) 1999-2001 Adrian E. Feiguin <adrian@ifir.ifir.edu.ar>
@@ -66,7 +72,6 @@
 /* sheet flags */
 enum
   {
-    GTK_SHEET_IS_LOCKED = 1 << 0,
     GTK_SHEET_IS_FROZEN = 1 << 1,
     GTK_SHEET_IN_XDRAG = 1 << 2,
     GTK_SHEET_IN_YDRAG = 1 << 3,
@@ -80,9 +85,6 @@ enum
 #define GTK_SHEET_SET_FLAGS(sheet,flag) (GTK_SHEET_FLAGS (sheet) |= (flag))
 #define GTK_SHEET_UNSET_FLAGS(sheet,flag) (GTK_SHEET_FLAGS (sheet) &= ~ (flag))
 
-#define GTK_SHEET_IS_LOCKED(sheet) (GTK_SHEET_FLAGS (sheet) & GTK_SHEET_IS_LOCKED)
-
-
 #define GTK_SHEET_IS_FROZEN(sheet) (GTK_SHEET_FLAGS (sheet) & GTK_SHEET_IS_FROZEN)
 #define GTK_SHEET_IN_XDRAG(sheet) (GTK_SHEET_FLAGS (sheet) & GTK_SHEET_IN_XDRAG)
 #define GTK_SHEET_IN_YDRAG(sheet) (GTK_SHEET_FLAGS (sheet) & GTK_SHEET_IN_YDRAG)
@@ -136,10 +138,13 @@ guint DEFAULT_ROW_HEIGHT (GtkWidget *widget)
        pango_context_get_metrics (context,
                                   widget->style->font_desc,
                                   pango_context_get_language (context));
+
       guint val = pango_font_metrics_get_descent (metrics) +
        pango_font_metrics_get_ascent (metrics);
+
       pango_font_metrics_unref (metrics);
-      return PANGO_PIXELS (val)+2 * CELLOFFSET;
+
+      return PANGO_PIXELS (val) + 2 * CELLOFFSET;
     }
 }
 
@@ -172,7 +177,7 @@ guint STRING_WIDTH (GtkWidget *widget,
 
   pango_layout_get_extents (layout, NULL, &rect);
 
-  g_object_unref (G_OBJECT (layout));
+  g_object_unref (layout);
   return PANGO_PIXELS (rect.width);
 }
 
@@ -1320,6 +1325,15 @@ range_update_callback (GSheetModel *m, gint row0, gint col0,
   range.rowi = rowi;
   range.coli = coli;
 
+  if ( MAX_VISIBLE_ROW (sheet) >
+       g_sheet_model_get_row_count (sheet->model)
+       ||
+       MAX_VISIBLE_COLUMN (sheet) >
+       g_sheet_model_get_column_count (sheet->model))
+    {
+      gtk_sheet_move_query (sheet, 0, 0);
+    }
+
   if ( ( row0 < 0 && col0 < 0 ) || ( rowi < 0 && coli < 0 ) )
     {
       gint i;
@@ -1440,7 +1454,7 @@ column_titles_changed (GtkWidget *w, gint first, gint n_columns, gpointer data)
       for ( i = first ; i <= first + n_columns ; ++i )
        {
          gtk_sheet_column_title_button_draw (sheet, i);
-         g_signal_emit (G_OBJECT (sheet), sheet_signals[CHANGED], 0, -1, i);
+         g_signal_emit (sheet, sheet_signals[CHANGED], 0, -1, i);
        }
     }
 
@@ -1536,7 +1550,7 @@ gtk_sheet_change_entry (GtkSheet *sheet, GtkType entry_type)
   if (state == GTK_SHEET_NORMAL)
     {
       gtk_sheet_show_active_cell (sheet);
-      g_signal_connect (G_OBJECT (gtk_sheet_get_entry (sheet)),
+      g_signal_connect (gtk_sheet_get_entry (sheet),
                        "changed",
                        G_CALLBACK (gtk_sheet_entry_changed),
                        sheet);
@@ -1740,40 +1754,6 @@ gtk_sheet_justify_entry (GtkSheet *sheet)
   return sheet->justify_entry;
 }
 
-void
-gtk_sheet_set_locked (GtkSheet *sheet, gboolean locked)
-{
-  g_return_if_fail (sheet != NULL);
-  g_return_if_fail (GTK_IS_SHEET (sheet));
-
-  if ( locked )
-    {
-      GTK_SHEET_SET_FLAGS (sheet, GTK_SHEET_IS_LOCKED);
-      gtk_widget_hide (sheet->sheet_entry);
-      gtk_widget_unmap (sheet->sheet_entry);
-    }
-  else
-    {
-      GTK_SHEET_UNSET_FLAGS (sheet, GTK_SHEET_IS_LOCKED);
-      if (GTK_WIDGET_MAPPED (GTK_WIDGET (sheet)))
-       {
-         gtk_widget_show (sheet->sheet_entry);
-         gtk_widget_map (sheet->sheet_entry);
-       }
-    }
-
-  gtk_editable_set_editable (GTK_EDITABLE (sheet->sheet_entry), locked);
-
-}
-
-gboolean
-gtk_sheet_locked (const GtkSheet *sheet)
-{
-  g_return_val_if_fail (sheet != NULL, FALSE);
-  g_return_val_if_fail (GTK_IS_SHEET (sheet), FALSE);
-
-  return GTK_SHEET_IS_LOCKED (sheet);
-}
 
 /* This routine has problems with gtk+- 1.2 related with the
    label / button drawing - I think it's a bug in gtk+- 1.2 */
@@ -1828,10 +1808,10 @@ gtk_sheet_thaw (GtkSheet *sheet)
   sheet->old_hadjustment = -1.;
 
   if (sheet->hadjustment)
-    g_signal_emit_by_name (G_OBJECT (sheet->hadjustment),
+    g_signal_emit_by_name (sheet->hadjustment,
                           "value_changed");
   if (sheet->vadjustment)
-    g_signal_emit_by_name (G_OBJECT (sheet->vadjustment),
+    g_signal_emit_by_name (sheet->vadjustment,
                           "value_changed");
 
   if (sheet->state == GTK_STATE_NORMAL)
@@ -1854,7 +1834,7 @@ gtk_sheet_set_row_titles_width (GtkSheet *sheet, guint width)
 
   sheet->old_hadjustment = -1.;
   if (sheet->hadjustment)
-    g_signal_emit_by_name (G_OBJECT (sheet->hadjustment),
+    g_signal_emit_by_name (sheet->hadjustment,
                           "value_changed");
   size_allocate_global_button (sheet);
 }
@@ -1870,7 +1850,7 @@ gtk_sheet_set_column_titles_height (GtkSheet *sheet, guint height)
 
   sheet->old_vadjustment = -1.;
   if (sheet->vadjustment)
-    g_signal_emit_by_name (G_OBJECT (sheet->vadjustment),
+    g_signal_emit_by_name (sheet->vadjustment,
                           "value_changed");
   size_allocate_global_button (sheet);
 }
@@ -1909,7 +1889,7 @@ gtk_sheet_show_column_titles (GtkSheet *sheet)
 
   sheet->old_vadjustment = -1.;
   if (sheet->vadjustment)
-    g_signal_emit_by_name (G_OBJECT (sheet->vadjustment),
+    g_signal_emit_by_name (sheet->vadjustment,
                           "value_changed");
   size_allocate_global_button (sheet);
 }
@@ -1951,7 +1931,7 @@ gtk_sheet_show_row_titles (GtkSheet *sheet)
 
   sheet->old_hadjustment = -1.;
   if (sheet->hadjustment)
-    g_signal_emit_by_name (G_OBJECT (sheet->hadjustment),
+    g_signal_emit_by_name (sheet->hadjustment,
                           "value_changed");
   size_allocate_global_button (sheet);
 }
@@ -1987,7 +1967,7 @@ gtk_sheet_hide_column_titles (GtkSheet *sheet)
 
   sheet->old_vadjustment = -1.;
   if (sheet->vadjustment)
-    g_signal_emit_by_name (G_OBJECT (sheet->vadjustment),
+    g_signal_emit_by_name (sheet->vadjustment,
                           "value_changed");
 }
 
@@ -2022,7 +2002,7 @@ gtk_sheet_hide_row_titles (GtkSheet *sheet)
 
   sheet->old_hadjustment = -1.;
   if (sheet->hadjustment)
-    g_signal_emit_by_name (G_OBJECT (sheet->hadjustment),
+    g_signal_emit_by_name (sheet->hadjustment,
                           "value_changed");
 }
 
@@ -2068,21 +2048,22 @@ gtk_sheet_moveto (GtkSheet *sheet,
   width = sheet->sheet_window_width;
 
   /* adjust vertical scrollbar */
-  if (row >= 0 && row_align >= 0.)
+  if (row >= 0 && row_align >= 0.0)
     {
       y = ROW_TOP_YPIXEL (sheet, row) - sheet->voffset
-       - (gint) ( row_align*height + (1. - row_align)
+       - (gint) ( row_align * height + (1.0 - row_align)
                   * yyy_row_height (sheet, row));
 
       /* This forces the sheet to scroll when you don't see the entire cell */
       min_row = row;
       adjust = 0;
-      if (row_align == 1.)
+      if (row_align >= 1.0)
        {
          while (min_row >= 0 && min_row > MIN_VISIBLE_ROW (sheet))
            {
              if (yyy_row_is_visible (sheet, min_row))
                adjust += yyy_row_height (sheet, min_row);
+
              if (adjust >= height)
                {
                  break;
@@ -2090,6 +2071,9 @@ gtk_sheet_moveto (GtkSheet *sheet,
              min_row--;
            }
          min_row = MAX (min_row, 0);
+
+         min_row ++;
+
          y = ROW_TOP_YPIXEL (sheet, min_row) - sheet->voffset +
            yyy_row_height (sheet, min_row) - 1;
        }
@@ -2100,23 +2084,22 @@ gtk_sheet_moveto (GtkSheet *sheet,
        sheet->vadjustment->value = y;
 
       sheet->old_vadjustment = -1.;
-      g_signal_emit_by_name (G_OBJECT (sheet->vadjustment),
+      g_signal_emit_by_name (sheet->vadjustment,
                             "value_changed");
 
     }
 
   /* adjust horizontal scrollbar */
-  if (column >= 0 && col_align >= 0.)
+  if (column >= 0 && col_align >= 0.0)
     {
       x = COLUMN_LEFT_XPIXEL (sheet, column) - sheet->hoffset
-       - (gint) ( col_align*width + (1.- col_align)*
+       - (gint) ( col_align*width + (1.- col_align)*
                   xxx_column_width (sheet, column));
 
-
       /* This forces the sheet to scroll when you don't see the entire cell */
       min_col = column;
       adjust = 0;
-      if (col_align == 1.)
+      if (col_align == 1.0)
        {
          while (min_col >= 0 && min_col > MIN_VISIBLE_COLUMN (sheet))
            {
@@ -2140,9 +2123,8 @@ gtk_sheet_moveto (GtkSheet *sheet,
        sheet->hadjustment->value = x;
 
       sheet->old_vadjustment = -1.;
-      g_signal_emit_by_name (G_OBJECT (sheet->hadjustment),
+      g_signal_emit_by_name (sheet->hadjustment,
                             "value_changed");
-
     }
 }
 
@@ -2212,7 +2194,7 @@ gtk_sheet_select_row (GtkSheet * sheet,
   sheet->active_cell.row = row;
   sheet->active_cell.col = 0;
 
-  g_signal_emit (G_OBJECT (sheet), sheet_signals[SELECT_ROW], 0, row);
+  g_signal_emit (sheet, sheet_signals[SELECT_ROW], 0, row);
   gtk_sheet_real_select_range (sheet, NULL);
 }
 
@@ -2243,7 +2225,7 @@ gtk_sheet_select_column (GtkSheet * sheet, gint column)
   sheet->active_cell.row = 0;
   sheet->active_cell.col = column;
 
-  g_signal_emit (G_OBJECT (sheet), sheet_signals[SELECT_COLUMN], 0, column);
+  g_signal_emit (sheet, sheet_signals[SELECT_COLUMN], 0, column);
   gtk_sheet_real_select_range (sheet, NULL);
 }
 
@@ -2346,21 +2328,21 @@ gtk_sheet_set_vadjustment (GtkSheet *sheet,
 
   if (sheet->vadjustment)
     {
-      g_signal_handlers_disconnect_matched (G_OBJECT (sheet->vadjustment),
+      g_signal_handlers_disconnect_matched (sheet->vadjustment,
                                            G_SIGNAL_MATCH_DATA,
                                            0, 0, 0, 0,
                                            sheet);
-      g_object_unref (G_OBJECT (sheet->vadjustment));
+      g_object_unref (sheet->vadjustment);
     }
 
   sheet->vadjustment = adjustment;
 
   if (sheet->vadjustment)
     {
-      g_object_ref (G_OBJECT (sheet->vadjustment));
-      g_object_ref_sink (G_OBJECT (sheet->vadjustment));
+      g_object_ref (sheet->vadjustment);
+      g_object_ref_sink (sheet->vadjustment);
 
-      g_signal_connect (G_OBJECT (sheet->vadjustment), "value_changed",
+      g_signal_connect (sheet->vadjustment, "value_changed",
                        G_CALLBACK (vadjustment_value_changed),
                        sheet);
     }
@@ -2392,21 +2374,21 @@ gtk_sheet_set_hadjustment (GtkSheet *sheet,
 
   if (sheet->hadjustment)
     {
-      g_signal_handlers_disconnect_matched (G_OBJECT (sheet->hadjustment),
+      g_signal_handlers_disconnect_matched (sheet->hadjustment,
                                            G_SIGNAL_MATCH_DATA,
                                            0, 0, 0, 0,
                                            sheet);
-      g_object_unref (G_OBJECT (sheet->hadjustment));
+      g_object_unref (sheet->hadjustment);
     }
 
   sheet->hadjustment = adjustment;
 
   if (sheet->hadjustment)
     {
-      g_object_ref (G_OBJECT (sheet->hadjustment));
-      g_object_ref_sink (G_OBJECT (sheet->hadjustment));
+      g_object_ref (sheet->hadjustment);
+      g_object_ref_sink (sheet->hadjustment);
 
-      g_signal_connect (G_OBJECT (sheet->hadjustment), "value_changed",
+      g_signal_connect (sheet->hadjustment, "value_changed",
                        G_CALLBACK (hadjustment_value_changed),
                        sheet);
     }
@@ -2480,23 +2462,23 @@ gtk_sheet_destroy (GtkObject * object)
   /* unref adjustments */
   if (sheet->hadjustment)
     {
-      g_signal_handlers_disconnect_matched (G_OBJECT (sheet->hadjustment),
+      g_signal_handlers_disconnect_matched (sheet->hadjustment,
                                            G_SIGNAL_MATCH_DATA,
                                            0, 0, 0, 0,
                                            sheet);
 
-      g_object_unref (G_OBJECT (sheet->hadjustment));
+      g_object_unref (sheet->hadjustment);
       sheet->hadjustment = NULL;
     }
 
   if (sheet->vadjustment)
     {
-      g_signal_handlers_disconnect_matched (G_OBJECT (sheet->vadjustment),
+      g_signal_handlers_disconnect_matched (sheet->vadjustment,
                                            G_SIGNAL_MATCH_DATA,
                                            0, 0, 0, 0,
                                            sheet);
 
-      g_object_unref (G_OBJECT (sheet->vadjustment));
+      g_object_unref (sheet->vadjustment);
 
       sheet->vadjustment = NULL;
     }
@@ -2704,7 +2686,7 @@ create_global_button (GtkSheet *sheet)
 {
   sheet->button = gtk_button_new_with_label (" ");
 
-  g_signal_connect (G_OBJECT (sheet->button),
+  g_signal_connect (sheet->button,
                    "pressed",
                    G_CALLBACK (global_button_clicked),
                    sheet);
@@ -2810,7 +2792,6 @@ gtk_sheet_map (GtkWidget * widget)
        }
 
       if (!GTK_WIDGET_MAPPED (sheet->sheet_entry)
-         && ! gtk_sheet_locked (sheet)
          && sheet->active_cell.row >= 0
          && sheet->active_cell.col >= 0 )
        {
@@ -3124,7 +3105,7 @@ gtk_sheet_cell_draw_label (GtkSheet *sheet, gint row, gint col)
                   layout);
 
   gdk_gc_set_clip_rectangle (fg_gc, NULL);
-  g_object_unref (G_OBJECT (layout));
+  g_object_unref (layout);
 
   gdk_draw_pixmap (sheet->sheet_window,
                   GTK_WIDGET (sheet)->style->fg_gc[GTK_STATE_NORMAL],
@@ -3466,7 +3447,7 @@ gtk_sheet_set_cell (GtkSheet *sheet, gint row, gint col,
     }
 
   if ( changed )
-    g_signal_emit (G_OBJECT (sheet), sheet_signals[CHANGED], 0, row, col);
+    g_signal_emit (sheet, sheet_signals[CHANGED], 0, row, col);
 
 }
 
@@ -3507,8 +3488,8 @@ gtk_sheet_real_cell_clear (GtkSheet *sheet, gint row, gint column)
     {
       g_sheet_model_datum_clear (model, row, column);
 
-      if (GTK_IS_OBJECT (sheet) && G_OBJECT (sheet)->ref_count > 0)
-       g_signal_emit (G_OBJECT (sheet), sheet_signals[CLEAR_CELL], 0,
+      if (G_OBJECT (sheet)->ref_count > 0)
+       g_signal_emit (sheet, sheet_signals[CLEAR_CELL], 0,
                       row, column);
     }
 
@@ -3769,8 +3750,8 @@ gtk_sheet_entry_changed (GtkWidget *widget, gpointer data)
 
   if (row < 0 || col < 0) return;
 
-  sheet->active_cell.row =1;
-  sheet->active_cell.col =1;
+  sheet->active_cell.row = -1;
+  sheet->active_cell.col = -1;
 
   text = gtk_entry_get_text (GTK_ENTRY (gtk_sheet_get_entry (sheet)));
 
@@ -3811,7 +3792,7 @@ gtk_sheet_deactivate_cell (GtkSheet *sheet)
   if ( sheet->active_cell.row == -1 || sheet->active_cell.col == -1 )
     return TRUE;
 
-  g_signal_handlers_disconnect_by_func (G_OBJECT (gtk_sheet_get_entry (sheet)),
+  g_signal_handlers_disconnect_by_func (gtk_sheet_get_entry (sheet),
                                        G_CALLBACK (gtk_sheet_entry_changed),
                                        sheet);
 
@@ -3851,16 +3832,6 @@ gtk_sheet_hide_active_cell (GtkSheet *sheet)
   gtk_sheet_get_attributes (sheet, row, col, &attributes);
   justification = attributes.justification;
 
-  if (text && strlen (text) != 0)
-    {
-      gtk_sheet_set_cell (sheet, row, col, justification, text);
-      g_signal_emit (G_OBJECT (sheet), sheet_signals[SET_CELL], 0, row, col);
-    }
-  else
-    {
-      gtk_sheet_cell_clear (sheet, row, col);
-    }
-
   row = sheet->active_cell.row;
   col = sheet->active_cell.col;
 
@@ -3893,12 +3864,9 @@ gtk_sheet_activate_cell (GtkSheet *sheet, gint row, gint col)
   g_return_val_if_fail (GTK_IS_SHEET (sheet), FALSE);
 
   if (row < 0 || col < 0) return FALSE;
-  if (row >= yyy_row_count (sheet) || col >= xxx_column_count (sheet))
-    return FALSE;
 
-  /* _gtkextra_signal_emit (GTK_OBJECT (sheet), sheet_signals[ACTIVATE], row, col, &veto);
-     if (!GTK_WIDGET_REALIZED (GTK_WIDGET (sheet))) return veto;
-  */
+  if ( row > yyy_row_count (sheet) || col > xxx_column_count (sheet))
+    return FALSE;
 
   if (!veto) return FALSE;
   if (sheet->state != GTK_SHEET_NORMAL)
@@ -3920,13 +3888,12 @@ gtk_sheet_activate_cell (GtkSheet *sheet, gint row, gint col)
 
   gtk_sheet_show_active_cell (sheet);
 
-
-  g_signal_connect (G_OBJECT (gtk_sheet_get_entry (sheet)),
+  g_signal_connect (gtk_sheet_get_entry (sheet),
                    "changed",
                    G_CALLBACK (gtk_sheet_entry_changed),
                    sheet);
 
-  _gtkextra_signal_emit (GTK_OBJECT (sheet), sheet_signals[ACTIVATE], row, col, &veto);
+  _gtkextra_signal_emit (GTK_OBJECT (sheet), sheet_signals [ACTIVATE], row, col, &veto);
 
   return TRUE;
 }
@@ -3972,10 +3939,6 @@ gtk_sheet_show_active_cell (GtkSheet *sheet)
 
   gtk_entry_set_visibility (GTK_ENTRY (sheet_entry), attributes.is_visible);
 
-  if (gtk_sheet_locked (sheet) || !attributes.is_editable)
-    gtk_editable_set_editable (GTK_EDITABLE (sheet_entry), FALSE);
-  else
-    gtk_editable_set_editable (GTK_EDITABLE (sheet_entry), TRUE);
 
   /*** Added by John Gotts. Mar 25, 2005 *********/
   old_text = gtk_entry_get_text (GTK_ENTRY (sheet_entry));
@@ -4372,7 +4335,7 @@ gtk_sheet_draw_border (GtkSheet *sheet, GtkSheetRange new_range)
 
   gdk_gc_set_clip_rectangle (sheet->xor_gc, &area);
 
-  for (i =1; i <= 1; ++i)
+  for (i = -1; i <= 1; ++i)
     gdk_draw_rectangle (sheet->sheet_window,
                        sheet->xor_gc,
                        FALSE,
@@ -4528,7 +4491,7 @@ gtk_sheet_real_select_range (GtkSheet * sheet,
 
   gtk_sheet_update_primary_selection (sheet);
 
-  g_signal_emit (G_OBJECT (sheet), sheet_signals[SELECT_RANGE], 0, &sheet->range);
+  g_signal_emit (sheet, sheet_signals[SELECT_RANGE], 0, &sheet->range);
 }
 
 
@@ -4552,8 +4515,6 @@ gtk_sheet_select_range (GtkSheet * sheet, const GtkSheetRange *range)
   if (range->col0 < 0 || range->coli < 0) return;
 
 
-  if ( gtk_sheet_locked (sheet)) return ;
-
   if (sheet->state != GTK_SHEET_NORMAL)
     gtk_sheet_real_unselect_range (sheet, NULL);
   else
@@ -4601,8 +4562,8 @@ gtk_sheet_real_unselect_range (GtkSheet * sheet,
   if (range->row0 < 0 || range->rowi < 0) return;
   if (range->col0 < 0 || range->coli < 0) return;
 
-  g_signal_emit (G_OBJECT (sheet), sheet_signals[SELECT_COLUMN], 0, -1);
-  g_signal_emit (G_OBJECT (sheet), sheet_signals[SELECT_ROW], 0, -1);
+  g_signal_emit (sheet, sheet_signals[SELECT_COLUMN], 0, -1);
+  g_signal_emit (sheet, sheet_signals[SELECT_ROW], 0, -1);
 
   if (gtk_sheet_range_isvisible (sheet, *range))
     gtk_sheet_draw_backing_pixmap (sheet, *range);
@@ -4720,23 +4681,23 @@ gtk_sheet_button_press (GtkWidget * widget,
 
   if (event->window == sheet->column_title_window)
     {
-      g_signal_emit (G_OBJECT (sheet),
+      g_signal_emit (sheet,
                     sheet_signals[BUTTON_EVENT_COLUMN], 0,
                     column, event);
 
       if ( event->type == GDK_2BUTTON_PRESS && event->button == 1)
-       g_signal_emit (G_OBJECT (sheet),
+       g_signal_emit (sheet,
                       sheet_signals[DOUBLE_CLICK_COLUMN], 0, column);
 
     }
   else if (event->window == sheet->row_title_window)
     {
-      g_signal_emit (G_OBJECT (sheet),
+      g_signal_emit (sheet,
                     sheet_signals[BUTTON_EVENT_ROW], 0,
                     row, event);
 
       if ( event->type == GDK_2BUTTON_PRESS && event->button == 1)
-       g_signal_emit (G_OBJECT (sheet),
+       g_signal_emit (sheet,
                       sheet_signals[DOUBLE_CLICK_ROW], 0, row);
     }
 
@@ -4843,7 +4804,6 @@ gtk_sheet_button_press (GtkWidget * widget,
       else if (sheet->cursor_drag->type == GDK_TOP_LEFT_ARROW &&
               !GTK_SHEET_IN_SELECTION (sheet)
               && ! GTK_SHEET_IN_DRAG (sheet)
-              && ! gtk_sheet_locked (sheet)
               && sheet->active_cell.row >= 0
               && sheet->active_cell.col >= 0
               )
@@ -4908,38 +4868,6 @@ gtk_sheet_button_press (GtkWidget * widget,
   return TRUE;
 }
 
-#if 0
-static gint
-gtk_sheet_scroll (gpointer data)
-{
-  GtkSheet *sheet;
-  gint x, y, row, column;
-  gint move;
-
-  sheet = GTK_SHEET (data);
-
-  GDK_THREADS_ENTER ();
-
-  gtk_widget_get_pointer (GTK_WIDGET (sheet), &x, &y);
-  gtk_sheet_get_pixel_info (sheet, x, y, &row, &column);
-
-  move = TRUE;
-
-  if (GTK_SHEET_IN_SELECTION (sheet))
-    gtk_sheet_extend_selection (sheet, row, column);
-
-  if (GTK_SHEET_IN_DRAG (sheet) || GTK_SHEET_IN_RESIZE (sheet))
-    {
-      move = gtk_sheet_move_query (sheet, row, column);
-      if (move) draw_xor_rectangle (sheet, sheet->drag_range);
-    }
-
-  GDK_THREADS_LEAVE ();
-
-  return TRUE;
-}
-#endif
-
 static void
 gtk_sheet_click_cell (GtkSheet *sheet, gint row, gint column, gboolean *veto)
 {
@@ -4988,7 +4916,7 @@ gtk_sheet_click_cell (GtkSheet *sheet, gint row, gint column, gboolean *veto)
       return;
     }
 
-  if (row == - 1 && column == - 1)
+  if (row == -1 && column == -1)
     {
       sheet->range.row0 = 0;
       sheet->range.col0 = 0;
@@ -5014,6 +4942,7 @@ gtk_sheet_click_cell (GtkSheet *sheet, gint row, gint column, gboolean *veto)
              *veto = FALSE;
              return;
            }
+         gtk_sheet_activate_cell (sheet, row, column);
        }
 
       if (gtk_sheet_autoscroll (sheet))
@@ -5058,7 +4987,7 @@ gtk_sheet_button_release (GtkWidget * widget,
       gtk_sheet_set_column_width (sheet, sheet->drag_cell.col,
                                  new_column_width (sheet, sheet->drag_cell.col, &x));
       sheet->old_hadjustment = -1.;
-      g_signal_emit_by_name (G_OBJECT (sheet->hadjustment), "value_changed");
+      g_signal_emit_by_name (sheet->hadjustment, "value_changed");
       return TRUE;
     }
 
@@ -5072,7 +5001,7 @@ gtk_sheet_button_release (GtkWidget * widget,
 
       gtk_sheet_set_row_height (sheet, sheet->drag_cell.row, new_row_height (sheet, sheet->drag_cell.row, &y));
       sheet->old_vadjustment = -1.;
-      g_signal_emit_by_name (G_OBJECT (sheet->vadjustment), "value_changed");
+      g_signal_emit_by_name (sheet->vadjustment, "value_changed");
       return TRUE;
     }
 
@@ -5097,7 +5026,7 @@ gtk_sheet_button_release (GtkWidget * widget,
       old_range = sheet->range;
       sheet->range = sheet->drag_range;
       sheet->drag_range = old_range;
-      g_signal_emit (G_OBJECT (sheet), sheet_signals[MOVE_RANGE], 0,
+      g_signal_emit (sheet, sheet_signals[MOVE_RANGE], 0,
                     &sheet->drag_range, &sheet->range);
       gtk_sheet_select_range (sheet, &sheet->range);
     }
@@ -5128,7 +5057,7 @@ gtk_sheet_button_release (GtkWidget * widget,
       sheet->drag_range = old_range;
 
       if (sheet->state == GTK_STATE_NORMAL) sheet->state = GTK_SHEET_RANGE_SELECTED;
-      g_signal_emit (G_OBJECT (sheet), sheet_signals[RESIZE_RANGE], 0,
+      g_signal_emit (sheet, sheet_signals[RESIZE_RANGE], 0,
                     &sheet->drag_range, &sheet->range);
       gtk_sheet_select_range (sheet, &sheet->range);
     }
@@ -5600,8 +5529,8 @@ gtk_sheet_move_query (GtkSheet *sheet, gint row, gint column)
 
   row_move = FALSE;
   column_move = FALSE;
-  row_align =1.;
-  col_align =1.;
+  row_align = -1.;
+  col_align = -1.;
 
   height = sheet->sheet_window_height;
   width = sheet->sheet_window_width;
@@ -5609,7 +5538,7 @@ gtk_sheet_move_query (GtkSheet *sheet, gint row, gint column)
   if (row >= MAX_VISIBLE_ROW (sheet) && sheet->state != GTK_SHEET_COLUMN_SELECTED)
     {
       row_align = 1.;
-      new_row = MIN (yyy_row_count (sheet), row + 1);
+      new_row = MIN (yyy_row_count (sheet) - 1, row + 1);
       row_move = TRUE;
       if (MAX_VISIBLE_ROW (sheet) == yyy_row_count (sheet) - 1 &&
          ROW_TOP_YPIXEL (sheet, yyy_row_count (sheet)- 1) +
@@ -5720,7 +5649,7 @@ gtk_sheet_entry_key_press (GtkWidget *widget,
                           GdkEventKey *key)
 {
   gboolean focus;
-  g_signal_emit_by_name (G_OBJECT (widget), "key_press_event", key, &focus);
+  g_signal_emit_by_name (widget, "key_press_event", key, &focus);
   return focus;
 }
 
@@ -5950,12 +5879,12 @@ gtk_sheet_key_press (GtkWidget *widget,
       break;
     case GDK_Home:
       row = 0;
-      while (!yyy_row_is_visible (sheet, row) && row < yyy_row_count (sheet)- 1) row++;
+      while (!yyy_row_is_visible (sheet, row) && row < yyy_row_count (sheet) - 1) row++;
       gtk_sheet_click_cell (sheet, row, sheet->active_cell.col, &veto);
       extend_selection = FALSE;
       break;
     case GDK_End:
-      row = yyy_row_count (sheet)- 1;
+      row = yyy_row_count (sheet) - 1;
       while (!yyy_row_is_visible (sheet, row) && row > 0) row--;
       gtk_sheet_click_cell (sheet, row, sheet->active_cell.col, &veto);
       extend_selection = FALSE;
@@ -6255,6 +6184,7 @@ gtk_sheet_size_allocate_entry (GtkSheet *sheet)
              style->fg[GTK_STATE_ACTIVE] = previous_style->fg[GTK_STATE_ACTIVE];
            }
          gtk_widget_set_style (GTK_WIDGET (sheet_entry), style);
+          g_object_unref (style);
        }
     }
 
@@ -6444,7 +6374,7 @@ create_sheet_entry (GtkSheet *sheet)
       gtk_widget_realize (sheet->sheet_entry);
     }
 
-  g_signal_connect_swapped (G_OBJECT (entry), "key_press_event",
+  g_signal_connect_swapped (entry, "key_press_event",
                            G_CALLBACK (gtk_sheet_entry_key_press),
                            sheet);
 
@@ -6639,7 +6569,7 @@ gtk_sheet_button_draw (GtkSheet *sheet, GdkWindow *window,
                                    "label",
                                    real_x, real_y,
                                    layout);
-                 g_object_unref (G_OBJECT (layout));
+                 g_object_unref (layout);
 
                  real_y += text_height + 2;
 
@@ -6695,7 +6625,7 @@ gtk_sheet_column_title_button_draw (GtkSheet *sheet, gint column)
 {
   GdkWindow *window = NULL;
   GdkRectangle allocation;
-  GtkSheetButton *button = NULL;
+
   gboolean is_sensitive = FALSE;
 
   if (!GTK_WIDGET_REALIZED (GTK_WIDGET (sheet))) return;
@@ -6724,7 +6654,7 @@ gtk_sheet_column_title_button_draw (GtkSheet *sheet, gint column)
     }
   else
     {
-      button = xxx_column_button (sheet, column);
+      GtkSheetButton *button = xxx_column_button (sheet, column);
       allocation.x = COLUMN_LEFT_XPIXEL (sheet, column) + CELL_SPACING;
       if (sheet->row_titles_visible)
        allocation.x -= sheet->row_title_area.width;
@@ -6734,6 +6664,12 @@ gtk_sheet_column_title_button_draw (GtkSheet *sheet, gint column)
       is_sensitive = xxx_column_is_sensitive (sheet, column);
       gtk_sheet_button_draw (sheet, window, button,
                             is_sensitive, allocation);
+
+      /* FIXME: Not freeing this button is correct (sort of),
+      because in PSPP the model always returns a static copy */
+
+      /* gtk_sheet_button_free (button); */
+
     }
 }
 
@@ -6784,7 +6720,7 @@ adjust_scrollbars (GtkSheet * sheet)
       sheet->vadjustment->step_increment = DEFAULT_ROW_HEIGHT (GTK_WIDGET (sheet));
       sheet->vadjustment->lower = 0;
       sheet->vadjustment->upper = SHEET_HEIGHT (sheet) + 80;
-      g_signal_emit_by_name (G_OBJECT (sheet->vadjustment), "changed");
+      g_signal_emit_by_name (sheet->vadjustment, "changed");
 
     }
 
@@ -6795,7 +6731,7 @@ adjust_scrollbars (GtkSheet * sheet)
       sheet->hadjustment->step_increment = DEFAULT_COLUMN_WIDTH;
       sheet->hadjustment->lower = 0;
       sheet->hadjustment->upper = SHEET_WIDTH (sheet)+ 80;
-      g_signal_emit_by_name (G_OBJECT (sheet->hadjustment), "changed");
+      g_signal_emit_by_name (sheet->hadjustment, "changed");
 
     }
 }
@@ -7091,7 +7027,7 @@ draw_xor_rectangle (GtkSheet *sheet, GtkSheetRange range)
 
   gdk_gc_set_clip_rectangle (sheet->xor_gc, &clip_area);
 
-  for (i =1; i <= 1; ++i)
+  for (i = -1; i <= 1; ++i)
     gdk_draw_rectangle (sheet->sheet_window,
                        sheet->xor_gc,
                        FALSE,
@@ -7198,8 +7134,8 @@ gtk_sheet_set_column_width (GtkSheet * sheet,
       gtk_sheet_range_draw (sheet, NULL);
     }
 
-  g_signal_emit (G_OBJECT (sheet), sheet_signals[CHANGED], 0, -1, column);
-  g_signal_emit (G_OBJECT (sheet), sheet_signals[NEW_COL_WIDTH], 0,
+  g_signal_emit (sheet, sheet_signals[CHANGED], 0, -1, column);
+  g_signal_emit (sheet, sheet_signals[NEW_COL_WIDTH], 0,
                 column, width);
 }
 
@@ -7231,8 +7167,8 @@ gtk_sheet_set_row_height (GtkSheet * sheet,
       gtk_sheet_range_draw (sheet, NULL);
     }
 
-  g_signal_emit (G_OBJECT (sheet), sheet_signals[CHANGED], 0, row, - 1);
-  g_signal_emit (G_OBJECT (sheet), sheet_signals[NEW_ROW_HEIGHT], 0,
+  g_signal_emit (sheet, sheet_signals[CHANGED], 0, row, - 1);
+  g_signal_emit (sheet, sheet_signals[NEW_ROW_HEIGHT], 0,
                 row, height);
 
 }
@@ -7633,11 +7569,14 @@ gtk_sheet_column_size_request (GtkSheet *sheet,
 {
   GtkRequisition button_requisition;
   GList *children;
+  GtkSheetButton *button = xxx_column_button (sheet, col);
 
   gtk_sheet_button_size_request (sheet,
-                                xxx_column_button (sheet, col),
+                                button,
                                 &button_requisition);
 
+  gtk_sheet_button_free (button);
+
   *requisition = button_requisition.width;
 
   children = sheet->children;
@@ -8114,12 +8053,12 @@ gtk_sheet_update_primary_selection (GtkSheet *sheet)
   if (!GTK_WIDGET_REALIZED (sheet))
     return;
 
-  clipboard = gtk_widget_get_clipboard (GTK_WIDGET (sheet), 
+  clipboard = gtk_widget_get_clipboard (GTK_WIDGET (sheet),
                                        GDK_SELECTION_PRIMARY);
-  
+
   if (gtk_sheet_range_isvisible (sheet, sheet->range))
     {
-      if (!gtk_clipboard_set_with_owner (clipboard, targets, 
+      if (!gtk_clipboard_set_with_owner (clipboard, targets,
                                         G_N_ELEMENTS (targets),
                                         primary_get_cb, primary_clear_cb,
                                         G_OBJECT (sheet)))