Re: [evolution-patches] fix for the bug #267330 [calendar]



On Wed, 2005-11-30 at 13:48 +0000, chen  wrote:
>  
> +       /* initialize */
> +       dtstart.tzid = NULL;
> +       dtstart.value = NULL;
> +       dtend.tzid = NULL;
> +       dtend.value = NULL;
> + 

I would suggest these changes in the e_cal_component_get_(dtstart/dtend)
APIs in e-d-s itself, instead of doing this initialization in all the
places.

Also, I guess, the general convention of any "get" method/function is to
return a proper value on success and "NULL"/equivalent on failure.

V. Varadhan



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]