Re: [Planner Dev] New patch for undo: Focus in - focus out undo

[Alvaro del Castillo <acs lambdaux com>]

El lun, 19-04-2004 a las 09:28, Mikael Hallendal escribió:
> On mån, 2004-04-19 at 06:10 +0000, Alvaro del Castillo wrote:
> 
> Hi,
> 
> > > +       g_value_set_string (&value, g_strdup (focus_in_name));
> > >
> > > This will leak, no need for the strdup.
> > 
> > Hmmm, g_value_unset doesn't free the string memory?
> 
> g_value_set_string does it's own g_strdup of the string, so you'll copy
> the string twice here but only free it once in g_value_unset.

Ok, I suppose it after richard comments. And it is the same that last
time I ask about freeing a gtk_tree_model so I hope I can apply the same
argument in all memory issues with GLib, GTK+ and GNOME libraries :)

Thanks Micke!

> 
> Regards,
>   Mikael Hallendal