TreeView stack

[Kevin Ryde <user42 zip com au>]

A couple more probable stack gremlins in TreeView.  The tooltip one is
pretty clear.  The column get_size depends on how that func is
implemented, but looking at the code of 2.12/14 it calls its renderers
(as opposed to returning pre-computed size).

--- GtkTreeView.xs      08 Oct 2008 09:22:12 +1100      1.49
+++ GtkTreeView.xs      08 Oct 2008 09:22:21 +1100      
@@ -899,8 +899,12 @@
        GtkTreePath *path = NULL;
        GtkTreeIter iter = {0, };
     PPCODE:
+       /* PUTBACK/SPAGAIN because gtk_tree_view_get_tooltip_context() calls
+          out to its model get_iter(), which may be perl */
+       PUTBACK;
        if (! gtk_tree_view_get_tooltip_context (tree_view, &x, &y, keyboard_tip, &model, &path, &iter))
                XSRETURN_EMPTY;
+       SPAGAIN;
        EXTEND (sp, 5);
        PUSHs (sv_2mortal (newSViv (x)));
        PUSHs (sv_2mortal (newSViv (y)));

--- GtkTreeViewColumn.xs        08 Oct 2008 09:22:56 +1100      1.32
+++ GtkTreeViewColumn.xs        08 Oct 2008 09:23:05 +1100      
@@ -360,7 +360,11 @@
        GdkRectangle cell_area = {0};
        gint x_offset = 0, y_offset = 0, width = 0, height = 0;
     PPCODE:
+       /* PUTBACK/SPAGAIN because gtk_tree_view_column_cell_get_size()
+          calls out to its renderers, which may be perl */
+       PUTBACK;
        gtk_tree_view_column_cell_get_size (tree_column, &cell_area, &x_offset, &y_offset, &width, &height);
+       SPAGAIN;
        EXTEND (sp, 5);
        PUSHs (sv_2mortal (newSViv (x_offset)));
        PUSHs (sv_2mortal (newSViv (y_offset)));


-- 
"It's too early."
"Mate, some of us have been up since 10:30!"