[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-pim
Subject:    Re: [Kde-pim] KMail crashes when message box is shown on startup
From:       Ingo =?iso-8859-15?q?Kl=F6cker?= <kloecker () kde ! org>
Date:       2007-06-18 19:06:14
Message-ID: 200706182106.16881 () erwin ! ingo-kloecker ! de
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Friday 15 June 2007 19:32, Thomas McGuire wrote:
> > > After a quick look at kmkernel.cpp I suggest to delay the
> > > instantiation of MailTransport::TransportManager by moving the
> > > connects from KMKernel's c'tor to the init() method. See the
> > > attached (and completely untested) patch.
> >
> > Thanks for answering. The patch works, besides the fact the the
> > message box pops up behind the mainwindow.
> >
> > However, there are still other message boxes (don't remember which
> > ones, but I've definitaly seen some) having the same problem.
> > Finding each of those message boxes and fixing them is not easily
> > possible.
> >
> > What we really need to do is to fix the KUniqueApplication thing:
> > Only create the new instance just before qApp->exec() in main.cpp,
> > not when the event loop is entered (which can happen before
> > qApp->exec() because of those message boxes).
> > Still, I have no idea how to do that.
>
> OK, attached a new patch, which will delay the call to newInstance
> until just before the main event loop if necessary. It is probably
> still not the nicest solution, but it should work in all situations.
>
> If nobody objects or has a better idea, I'll commit that soon.

It doesn't look pretty, but if it works then I guess it should be=20
committed. Thanks for looking into this.

Regards,
Ingo

[Attachment #5 (application/pgp-signature)]

_______________________________________________
kde-pim mailing list
kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
kde-pim home page at http://pim.kde.org/

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic