[Development] Cleanup of QCoreApplication::watchUnixSignal

[Rafael Roquetto]

Hi,

On Fri, Oct 26, 2012 at 12:02:43PM +0200, Paul Olav Tvete wrote:
> On Thursday 25 October 2012 12:16:40 Rafael Roquetto wrote:
> > Afaik QCoreApplication::watchUnixSignal() seems to be no longer used, at
> > least in Qt5. If that is really the case, would anyone object doing away
> > with it (and removing the overhead from QEventDispatchUnix::doSelect() and
> > co.)? Otherwise, what are the possible use cases?
> > 
> > Any thoughts?
> 
> As the original author, I agree that it should be removed. Since 
> watchUnixSignal() is not implemented for the GLib event dispatcher, developers 
> cannot rely on it being available. Also, the original use case is no longer 
> relevant: it was used by QWS to detect virtual console switching.
> 
> The recommended way to watch for UNIX signals in Qt is:
> http://doc-snapshot.qt-project.org/5.0/unix-signals.html
> (making a signal handler that writes to a socket). 

Thanks everyone for the feedback. I will wait until the weekend and then if no
one opposes, I will submitt a patch to gerrit.

- Rafael

-- 
** Qt Developer Conference: http://qtconference.kdab.com/ **

Rafael Roquetto | rafael.roquetto at kdab.com | Software Engineer
Klarälvdalens Datakonsult AB, a KDAB Group company
Tel. Sweden (HQ) +46-563-540090, USA +1-866-777-KDAB(5322)
KDAB - Qt Experts - Platform-independent software solutions

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3721 bytes
Desc: not available
URL: <http://lists.qt-project.org/pipermail/development/attachments/20121026/7ed3964f/attachment.bin>