Currently when logging out, kio_http_cache_cleaner stays alive and blocks in
QCoreApplication::processEvents(QEventLoop::WaitForMoreEvents);
forever.
This patch avoids this blocking and ensures that the main loop will exit when the session bus is no longer connected.
BUG: 367575
FIXED-IN: 5.54
logging in, check CPU load of kio_http_cache_cleaner, log out, check if kio_http_cache_cleaner still runs.
| Lint Skipped |
| Unit Tests Skipped |
I assume the processEvents() is necessary to process DBus disconnection, then?
In that case let's add a comment, so the next person doesn't think this is unnecessary (it's certainly not necessary for the QLocalServer/QLocalSocket stuff).
David is right, processEvents() seems not be needed at all - removed it.
Actually while testing, I found some other bugs - mainly read/write of the scoreboard and the info from the slaves
where one side wrote qint64 but the other interpreted it as a QDateTime or the file size was expected to be 64bit in
one code part, but on another code part it was assumed to be 32bit...