[vlc-devel] [PATCH 1/3] Qt: Run mainloop on the main thread on OSX

Rémi Denis-Courmont remi at remlab.net
Sat Jul 31 20:00:22 CEST 2010


Le samedi 31 juillet 2010 20:44:52 Juho Vähä-Herttua, vous avez écrit :
> Thanks for your comments. It would be useful to know why they look wrong,
> that way I don't have to iterate until they "look right". I've already had
> to go to the commit history and ask around to find reasons for some
> changes, because the code is mainly documented with "/* */" if anything.

AFAICT, the semaphore is there because [other platforms] initialize Qt4 in a 
different thread.

Anyway, by design of a semaphore (in general), there is absolutely no point in 
a semaphore that is only ever used from a single thread. It will either result 
in a no-op or a deadlock, neither of which is useful.

-- 
Rémi Denis-Courmont
http://www.remlab.net/
http://fi.linkedin.com/in/remidenis



More information about the vlc-devel mailing list