~neon/project-neon/polkit-kde-agent-1

Viewing all changes in revision 326.

  • Committer: Friedrich W. H. Kossebau
  • Date: 2018-03-20 23:27:50 UTC
  • Revision ID: git-v1:576d10209bd95cce08c23c078be5b174e4bf3ac6
Fix app initialization order

Summary:
Patch inlines the code of the constructor and destructor of the old
QApplication subclass PolicyKitKDE and then reorders things as needed:
* KDBusService for singleton run should be started before any further init
* KLocalizedString::setApplicationDomain & i18n calls should be only done
  after the QApp instance has been created
* KAboutData::setApplicationData best done before doing any D-Bus activity
  as that relies on app metadata
* QCoreApplication::setOrganizationDomain duplicates app metadata
  setting from KAboutData::setApplicationData, only needed before due to
  latter done after first D-Bus activity
* m_listener->deleteLater() in app destructor does not make sense, as
  the listener object is deleted due to being QObject child of the
  QApplication instance

Reviewers: davidedmundson

Reviewed By: davidedmundson

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D11106

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: