Re: [Evolution-hackers] [Camel] When is the store "finalize()" called?

[Not Zed <notzed ximian com>]

Ok, good.  Well, it would be ice to get it fixed at some point, it
should probably be on the debug list.  If it got fixed and it was a
small patch it should be backported to 2.4 i'd suggest - although i dont
know what the plans are now.

On Wed, 2005-09-14 at 10:04 +0200, Jules Colding wrote:
> On Tue, 2005-09-13 at 22:17 +0800, Not Zed wrote:
> > How important is it?
> 
> Not really that important, more in the line of "If it was designed to be
> called then it should be called"-issue. Nitpicking i guess.
> 
> > Exactly what are you requiring it for?
> 
> Freeing local resources.
> 
> > If you're using it to free remote resources, like connections, then this
> > is the wrong place to do it.  disconnect() is the place for these types
> > of resource deallocations.
> 
> OK, I do that already.
> 
> > Otherwise, if it is just memory leaks, it just isn't important.
> 
> It is only due to memory leaks.
> 
> Regards,
>   jules
> 
> 
> 
> > On Tue, 2005-09-13 at 12:00 +0200, Jules Colding wrote:
> > > On Tue, 2005-09-13 at 17:24 +0800, Not Zed wrote:
> > > > No.
> > > > 
> > > > Sync could be called at any time.
> > > > 
> > > > I wasn't aware it was a known issue either, stores certainly used to be
> > > > finalised at one point.
> > > 
> > > Is this an issue that will be fixed in the 2.4 branch or has it to low
> > > of a priority to fix unless there isn't anything else to hack on?
> > > 
> > > Thanks,
> > >   jules
> > > 
> > > 
> > > 
> > > > On Tue, 2005-09-13 at 11:24 +0200, Jules Colding wrote:
> > > > > On Tue, 2005-09-13 at 03:15 -0600, Susarla Parthasarathi wrote:
> > > > > > Thats a known issue. A little complicated to fix. But is surely fixable.
> > > > > > 
> > > > > > So, none of the providers have their finalize method called.
> > > > > 
> > > > > OK, thanks. I can see that the last method being called is sync().
> > > > > Could/should I safely call finalize() from that one?
> > > > > 
> > > > > Thanks,
> > > > >   jules
> > > > > 
> > > > > 
> > > > > 
> > > > > _______________________________________________
> > > > > Evolution-hackers mailing list
> > > > > Evolution-hackers gnome org
> > > > > http://mail.gnome.org/mailman/listinfo/evolution-hackers
> > > 
> 
-- 
adfa(evolution-2.4:20087): gtkhtml-WARNING **: cannot find icon:
'stock_insert-url' in gnome