Re: [evolution-patches] More fixes for the evolution-data-server memory buildup [Calendar]
- From: Rodrigo Moya <rodrigo novell com>
- To: Harish Krishnaswamy <kharish novell com>
- Cc: Evolution Patches <evolution-patches lists ximian com>
- Subject: Re: [evolution-patches] More fixes for the evolution-data-server memory buildup [Calendar]
- Date: Wed, 08 Dec 2004 16:06:34 +0100
On Wed, 2004-12-08 at 18:57 -0800, Harish Krishnaswamy wrote:
> hi,
>
> The attached patch provides a few more fixes - basically the EList held
> by e-cal-backend.c not getting freed and thereby not letting
> edatacalview objects to get freed , holding the ecalbackends .... ( a
> classic case of domino effect)
>
> kindly review the same.
>
> @@ -2581,7 +2581,7 @@ e_cal_get_objects_for_uid (ECal *ecal, c
> comp = e_cal_component_new ();
> e_cal_component_set_icalcomponent (comp, icalcomponent_new_clone (subcomp));
> *objects = g_list_append
> (*objects, comp);
> -
> + icalcomponent_free (subcomp);
> subcomp =
> icalcomponent_get_next_component (icalcomp, kind_to_find);
> }
this call to icalcomponent_free should not be needed, since we are
freeing the full VCALENDAR after the while loop.
the rest looks ok
--
Rodrigo Moya <rodrigo novell com>
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]