Review Request: Replace thread usage with local event loop in kio/kio/hostinfo.cpp

Albert Astals Cid aacid at kde.org
Tue Aug 9 14:45:05 BST 2011


On Dimarts 09 Agost 2011 12:14:53 Thomas Zander wrote:
> On Tuesday 09 August 2011 12.59.49 Thomas Zander wrote:
> > Another solution for using a timeout can be something like the
> > attached version (I didn't even try to compile it, maybe the helper
> > object has to be
> > moved to a _p.h file to get moc to run on it..)
> 
> Oh, and the mutex has to be made recursive in the constructor.  Sorry, just
> coded this in my lunch break...

This won't work since your main thread is locked in the mutex thus the signal 
will not be delivered since there is no event processing happening.

Albert




More information about the kde-core-devel mailing list