D6711: Check for HUP/ERR/NVAL conditions on stdin
Fabian Vogt
noreply at phabricator.kde.org
Sat Jul 15 12:05:49 UTC 2017
fvogt created this revision.
Restricted Application added a project: Plasma.
REVISION SUMMARY
While Qt's event loop checks for POLLHUP, it
treats it exactly like POLLIN and it's not possible
to retrieve that information from Qt. So in the
Connection read handler it needs to be checked for
again.
TEST PLAN
Before this patch plasma-browser-integration-host
consumed 100% CPU after exiting the browser, 100% reproducible.
Now it quits gracefully.
REPOSITORY
R856 Plasma Browser Integration
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D6711
AFFECTED FILES
host/connection.cpp
To: fvogt, #plasma, broulik, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170715/21443c8e/attachment.html>
More information about the Plasma-devel
mailing list