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

List:       kde-commits
Subject:    Re: KDE/kdepim
From:       David Faure <dfaure () kdab ! net>
Date:       2009-01-08 16:40:23
Message-ID: 200901081740.24039.dfaure () kdab ! net
[Download RAW message or body]

On Wednesday 07 January 2009, Thomas McGuire wrote:
> +#ifdef Q_WS_MACX
> + // ### This quits the application prematurely, for example when the \
> composer window + // ### is closed while the main application is \
> minimized to the systray +  connect( qApp, SIGNAL(lastWindowClosed()), \
> qApp, SLOT(quit())); +#endif

Yep, the comment is right, this is exactly why you should not have such a \
connect in kmail or kontact... Instead, KDE apps use the KGlobal::ref/deref \
mechanism. Each user-visible thing that should keep the app alive does a \
ref on creation and a deref on destruction, like kmainwindows, systray \
icons, running kio jobs, etc. More info in the apidox for KGlobal::ref().

If the app doesn't exit, then you'll have to debug where the remaining \
ref(s) are. No idea off hand why it would be different on the mac though.
Note that the kdelibs code for this (kglobal, kmainwindow and ksystemtray) \
has been changed until a few months ago, so it would be worth just removing \
the connect and trying again with kdelibs-4.2 or trunk.

-- 
David Faure, faure@kde.org, dfaure@kdab.net
KDE/KOffice developer, Qt consultancy projects
Klarälvdalens Datakonsult AB, Platform-independent software solutions


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