Re: [evolution-patches] patch for #41579
- From: Rodrigo Moya <rodrigo ximian com>
- To: Hans Petter Jansson <hpj ximian com>
- Cc: Evolution Patches <evolution-patches ximian com>
- Subject: Re: [evolution-patches] patch for #41579
- Date: 14 May 2003 23:36:18 +0200
On Wed, 2003-05-14 at 22:23, Hans Petter Jansson wrote:
> On Wed, 2003-05-14 at 07:21, Rodrigo Moya wrote:
> > I can't replicate the bug, since the calendar file loads correctly both
> > in local and connector calendars. Also, double clicking on it opens up
> > the editor correctly, with no crashes.
> >
> > So, I can't track down why there's a NULL property, thus this fix only
> > makes sure we don't try to add the NULL property.
>
> Looks good, apart from nitpick:
>
> > + if (p != 0)
> > + icalcomponent_add_property(new,icalproperty_new_clone(p));
>
> Should be (p != NULL)?
>
yes, but libical uses 0 always, AFAIK. That's why I used 0 instead of
NULL.
cheers
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]