Re: [Vala] TreePath from TreeView.get_cursor() never freed?

[Geert Jan <gj_alsem yahoo com>]

Cool, thanks for the explanation and bugfix. If I encounter something similar I'll post it on Bugzilla 
instead next time.

Geert Jan


--- On Sun, 10/12/08, Hans Vercammen <hveso3 gmail com> wrote:

> From: Hans Vercammen <hveso3 gmail com>
> Subject: Re: [Vala] TreePath from TreeView.get_cursor() never freed?
> To: gj_alsem yahoo com
> Cc: vala-list gnome org
> Date: Sunday, October 12, 2008, 4:26 AM
> On Sat, 2008-10-11 at 14:50 -0700, Geert Jan wrote:
> > When I don't use a weak TreePath, like this:
> >
> >   TreePath path;
> >   weak TreeViewColumn focus_column;
> >   treeview.get_cursor (out path, out focus_column);
> >
> > I get this error:
> >
> > error: duplicating TreePath instance, use weak
> variable or explicitly invoke copy method
> >     treeview.get_cursor (out path, out focus_column);
> >                          ^^^^^^^^
>
> Yes, you should use a strong reference for the Gtk.TreePath
> parameter.
> This is a bug[1] in the gtk+-2.0.vapi file.
>
> The current signature in Gtk.TreeView reads:
>       public void get_cursor (out weak Gtk.TreePath path, out
> weak
> Gtk.TreeViewColumn focus_column);
>       
> While it should be something like:
>       public void get_cursor (out Gtk.TreePath? path, out weak
> Gtk.TreeViewColumn? focus_column);
>
> The vala api files for GObject based libraries are
> automatically
> generated through introspection. But sometimes there is not
> enough
> information on how to use the parameters correctly, so
> these additional
> attributes are added in a metafile as described in the
> wiki[2]. For most
> api's it should be ok, but it's always possible
> some parameters need
> additional information which isn't there yet. If you
> encounter similar
> problems feel free to file a bug at
> http://bugzilla.gnome.org/.
>
> Hans
>
> [1] http://bugzilla.gnome.org/show_bug.cgi?id=555972
> [2] http://live.gnome.org/Vala/Bindings