[Kde-pim] [patch] kdepimlibs/mailtransport: destroy TransportManager using qAddPostRoutine

[Tom Albers]

Op maandag 28 juli 2008 09:18 schreef u:
> On Sunday 27 July 2008 17:30:23 Tom Albers wrote:
> > Op woensdag 18 juni 2008 11:52 schreef u:
> > > On Wednesday 18 June 2008 11:45:34 Tom Albers wrote:
> > > > As beta 2 Op woensdag 18 juni 2008 11:37 schreef u:
> > > > > (for review; enterprise4 branch)
> > > > >
> > > > > Destroy TransportManager using qAddPostRoutine, not K_GLOBAL_STATIC.
> > > > > This avoids crash on application's quit: destroying through
> > > > > K_GLOBAL_STATIC is too late for this object.
> > > >
> > > > I don't see a crash on applications quit.
> > > >
> > > > As this is too late for beta2, which is the last beta, I ask to wait
> > > > with this patch for KDE 4.2.
> > >
> > > Please commit it to enterprise4 branch, we can test it there, and merge
> > > it back with the rest of the stuff after the un-freeze.
> > >
> > > Till
> >
> > Your friendly reminder
> 
> Thomas McGuire should get to it as part of normal merging. It was committed to 
> e4, right?
> 

Just replying here, I lost Thomas' mail. The commit in revision 822818, mean that it is committed to trunk before it was branched of, right? If so, it is part of 4.1.0 now, which is against what I requested/we decided above. 

Too bad I missed the commit ;-) A great opportunity to yell had gone now ;-)

Best.

Toma

-------------- next part --------------
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/