D8673: Sanitize signal handling in ksmserver

Martin Flöser noreply at phabricator.kde.org
Thu Nov 30 19:02:23 UTC 2017


graesslin added inline comments.

INLINE COMMENTS

> server.cpp:615
> +    QSocketNotifier* n = new QSocketNotifier(sockets[1], QSocketNotifier::Read, this);
> +    qApp->connect(n, SIGNAL(activated(int)), SLOT(quit()));
> +

why the old connect syntax?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D8673

To: jpalecek, #plasma
Cc: graesslin, davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171130/911679aa/attachment.html>


More information about the Plasma-devel mailing list