Revision 1240914

Go back to digest for 17th July 2011

Bug Fixes in Utilities

Raphael Kubo da Costa committed changes in /trunk/KDE/kdeutils/ark/kerfuffle:

Do not connect to KJob::result() from the worker thread.

This is another step towards getting rid of the threading mess we
currently have.

We previously connected to KJob::result() from the worker thread using
Qt::DirectConnection just to make sure QThread::quit() was called
right away (synchronously). We now make the connection from the main
thread without Qt::DirectConnection, and just make sure we start the
worker thread's event loop if there is an actual need to.

The other use of Qt::DirectConnection in the worker thread (via
QMetaObject::invkeMethod) has also been removed, as it was there to
make sure the method was called right away from the worker thread as
well.

File Changes

Modified 2 files
  • /trunk/KDE/kdeutils/ark/kerfuffle
  •   /jobs.cpp
  •   /jobs.h
2 files changed in total