Re: [evolution-patches] Patch to allow an appointment/meeting to be opened in the editor from command-line.

[Harish <kharish novell com>]

hi,

On Thu, 2005-07-07 at 22:17 +0530, Veerapuram Varadhan wrote:
>     /* FIXME: Do I need to free this icalcomp? */
>     +       e_cal_get_object (client, comp_uid, comp_rid, &icalcomp, NULL);
>     +
you need to add some error handling here.
>     +       attendee_prop = icalcomponent_get_first_property (icalcomp, ICAL_ATTENDEE_PROPERTY);
>     +       e_calendar_view_edit_appointment (gcal->priv->views[gcal->priv->current_view_type],
>     +                                         client, icalcomp, attendee_prop ? TRUE:FALSE); 
the above call does a clone of the argument you send. so yes - you need to free the
icalcomp.
Pl. commit the same after adding the above changes.

Harish