https://bugs.kde.org/show_bug.cgi?id=382445
Fabian Vogt <fab...@ritter-vogt.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |RESOLVED
Resolution|--- |FIXED
Latest Commit| |https://commits.kde.org/pla
| |sma-browser-integration/4c2
| |a3b16803677fe9a7f22e3cda725
| |5cfd140815
--- Comment #2 from Fabian Vogt <fab...@ritter-vogt.de> ---
Git commit 4c2a3b16803677fe9a7f22e3cda7255cfd140815 by Fabian Vogt.
Committed on 18/07/2017 at 08:35.
Pushed by fvogt into branch 'master'.
Check for HUP/ERR/NVAL conditions on stdin
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.
Reviewers: #plasma, broulik, davidedmundson
Subscribers: plasma-devel
Tags: #plasma
Differential Revision: https://phabricator.kde.org/D6711
M +19 -0 host/connection.cpp
https://commits.kde.org/plasma-browser-integration/4c2a3b16803677fe9a7f22e3cda7255cfd140815
--
You are receiving this mail because:
You are watching all bug changes.
