Re: [evolution-patches] patch for #41579



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]