Need suggestion on how to fix the common crash in plasma-desktop (kdelibs related)

2012-03-20 Thread Lamarque V. Souza
Hi, There is a crash in WeatherEngine (kde-workspace) triggered by the fact that Plasma::DataEngineManager::self() (kdelib) is invalid when plasma-{desktop,device} are exiting. WeatherEngine::~WeatherEngine() calls WeatherEngine::unloadIons(), which tries to use the invalid Pla

Re: Need suggestion on how to fix the common crash in plasma-desktop (kdelibs related)

2012-03-20 Thread Thomas Lübking
Tried to connect to QCoreApplication::aboutToQuit()? Am 21.03.2012, 02:31 Uhr, schrieb Lamarque V. Souza : Hi, There is a crash in WeatherEngine (kde-workspace) triggered by the fact that Plasma::DataEngineManager::self() (kdelib) is invalid when plasma-{desktop,device} are exi

Re: Need suggestion on how to fix the common crash in plasma-desktop (kdelibs related)

2012-03-20 Thread Lamarque V. Souza
Em Tuesday 20 March 2012, Thomas Lübking escreveu: > Tried to connect to QCoreApplication::aboutToQuit()? It is already done but that does not cover the case where WeatherEngine is unloaded without plasma-{desktop,device} exiting. The Ions must be unloaded in that case or we will have a

Re: Need suggestion on how to fix the common crash in plasma-desktop (kdelibs related)

2012-03-21 Thread Thomas Lübking
hook on QObject::destroyed() for the dataenginemanager (and flag yourself "dontTry")?

Re: Need suggestion on how to fix the common crash in plasma-desktop (kdelibs related)

2012-03-21 Thread David Faure
On Tuesday 20 March 2012 22:31:57 Lamarque V. Souza wrote: > WeatherEngine::~WeatherEngine() calls > WeatherEngine::unloadIons(), which tries to use the invalid > Plasma::DataEngineManager::self(). Since DataEngineManager uses K_GLOBAL_STATIC internally, just use the isDestroyed() method to kno

Re: Need suggestion on how to fix the common crash in plasma-desktop (kdelibs related)

2012-03-21 Thread Aaron J. Seigo
On Tuesday, March 20, 2012 22:31:57 Lamarque V. Souza wrote: > There is a crash in WeatherEngine (kde-workspace) triggered by the fact > that Plasma::DataEngineManager::self() (kdelib) is invalid when > plasma-{desktop,device} are exiting. WeatherEngine::~WeatherEngine() calls > WeatherEngine

Re: Need suggestion on how to fix the common crash in plasma-desktop (kdelibs related)

2012-03-21 Thread Lamarque V. Souza
Em Wednesday 21 March 2012, Thomas Lübking escreveu: > hook on QObject::destroyed() for the dataenginemanager (and flag > yourself "dontTry")? Didn't try it Unfortunately it also did not work :( WeatherEngine did not receive the signal, at least not before the crash. Thanks anywa

Re: Need suggestion on how to fix the common crash in plasma-desktop (kdelibs related)

2012-03-21 Thread Lamarque V. Souza
Em Wednesday 21 March 2012, David Faure escreveu: > On Tuesday 20 March 2012 22:31:57 Lamarque V. Souza wrote: > > WeatherEngine::~WeatherEngine() calls > > WeatherEngine::unloadIons(), which tries to use the invalid > > Plasma::DataEngineManager::self(). > > Since DataEngineManager uses K_GLOBAL_

Re: Need suggestion on how to fix the common crash in plasma-desktop (kdelibs related)

2012-03-21 Thread Lamarque V. Souza
Em Wednesday 21 March 2012, Aaron J. Seigo escreveu: > On Tuesday, March 20, 2012 22:31:57 Lamarque V. Souza wrote: > > There is a crash in WeatherEngine (kde-workspace) triggered by the fact > > > > that Plasma::DataEngineManager::self() (kdelib) is invalid when > > plasma-{desktop,device} ar

Re: Need suggestion on how to fix the common crash in plasma-desktop (kdelibs related)

2012-03-21 Thread David Faure
On Wednesday 21 March 2012 09:06:05 Lamarque V. Souza wrote: > Em Wednesday 21 March 2012, David Faure escreveu: > > On Tuesday 20 March 2012 22:31:57 Lamarque V. Souza wrote: > > > WeatherEngine::~WeatherEngine() calls > > > WeatherEngine::unloadIons(), which tries to use the invalid > > > Plasma:

Re: Need suggestion on how to fix the common crash in plasma-desktop (kdelibs related)

2012-03-22 Thread Lamarque V. Souza
Em Wednesday 21 March 2012, Lamarque V. Souza escreveu: > Em Wednesday 21 March 2012, Aaron J. Seigo escreveu: > > On Tuesday, March 20, 2012 22:31:57 Lamarque V. Souza wrote: > > > There is a crash in WeatherEngine (kde-workspace) triggered by the > > > fact > > > > > > that Plasma::DataEngin