[Differential] [Request, 5 lines] D3146: Manually delete PowerDevil Core on teardown

broulik (Kai Uwe Broulik) noreply at phabricator.kde.org
Sun Oct 23 17:48:15 UTC 2016


broulik created this revision.
broulik added a reviewer: Plasma.
broulik set the repository for this revision to rPOWERDEVIL Powerdevil.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  The QObject children cleanup routine is run too late at a point where the QGuiApplication's X connection is already cleaned up. By manually deleting in PowerDevilApp's destructor, we delete Core first before any Qt cleanup happens.
  
  BUG: 371127
  FIXED-IN: 5.8.3

TEST PLAN
  PowerDevil no longer crashes when quitting it:
  
    qdbus local.org_kde_powerdevil /MainApplication org.qtproject.Qt.QCoreApplication.quit

REPOSITORY
  rPOWERDEVIL Powerdevil

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

AFFECTED FILES
  daemon/powerdevilapp.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161023/e3d1d00a/attachment.html>


More information about the Plasma-devel mailing list