Skip to content
Commit fd719f72 authored by Bernhard Beschow's avatar Bernhard Beschow Committed by Dennis Nienhüser
Browse files

instantiate event loops in runner plugins rather than in RunnerTask and...

instantiate event loops in runner plugins rather than in RunnerTask and instantiate runners in background threads

Instantiating the event loops in the runner plugins rather than in RunnerTask resolves weired usage of QEventLoop: The synchronous runners were finished before QEventLoop::exec() was called. Somehow the finished signal of the runners still managed to quit the loop, such that exec() wouldn't block. Until Qt 4.8 we might just have been lucky...

Instantiating runners in the background thread avoids thread affinity issues wrt. to QEventLoop. In particular, QEventLoop instances would block a background thread forever because calling quit() or exit() wouldn't cause exec() to return.

BUG: 288612
FIXED-IN: 1.3.0
REVIEW: 103711
parent 5b9e36dd
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment