D18245: [DrKonqi] Create QApplication in a scope to fix possible deadlock
Christoph Roick
noreply at phabricator.kde.org
Fri Feb 22 22:21:52 GMT 2019
croick added a subscriber: ossi.
croick added a comment.
I dug in the history of KCrash and realized, that closing all the file descriptors was introduced (7bbf112e5e97d40b9a4f9fdd167b064a1019843b <https://phabricator.kde.org/R446:7bbf112e5e97d40b9a4f9fdd167b064a1019843b>) to make the X11 window disappear.
Later @ossi added the X11 FD explicitly (7ac326ddb05eec296fdfbebd86d0d1320e579063 <https://phabricator.kde.org/R446:7ac326ddb05eec296fdfbebd86d0d1320e579063>), but kept closing the FDs by default.
I think keeping the FDs should be the default. However, the only program I found that is explicitly setting `KCrash::CrashFlag::KeepFDs` is #falkon <https://phabricator.kde.org/tag/falkon/>, the rest seems to be fine without (except me with KDevelop).
REPOSITORY
R871 DrKonqi
REVISION DETAIL
https://phabricator.kde.org/D18245
To: croick, sitter, #kdevelop
Cc: ossi, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190222/53dde328/attachment.html>
More information about the Plasma-devel
mailing list