----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124902/ -----------------------------------------------------------
(Updated Aug. 25, 2015, 5:58 a.m.) Status ------ This change has been marked as submitted. Review request for KDE Frameworks and Kevin Funk. Changes ------- Submitted with commit 40d047f013dc6f0e2f78066184a51a8c6e22eccd by Martin Gräßlin to branch master. Repository: kglobalaccel Description ------- There are usages of KGlobalAccel in e.g. KActionCollection of xmlgui framework which do not require a running kglobalaccel5. But due to the way KGlobalAccel inits, it always started the runtime part. This change ensures that kglobalaccel5 is only started once it's actually needed, by e.g. registering a global shortcut. If an application does not use global shortcuts the API should no longer trigger a start of the runtime part. Diffs ----- src/kglobalaccel.cpp 3508c870c70a21e76263b7574c20b408cc09a837 src/kglobalaccel_p.h a84f4038dac650ddb0fc1828e67e0851153b4456 Diff: https://git.reviewboard.kde.org/r/124902/diff/ Testing ------- export $(dbus-launch) qdbusviewer & kate & -> kglobalaccel5 gets started with the change: kglobalaccel5 no longer gets started, when starting an application using global shortcuts, it still gets started Thanks, Martin Gräßlin
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel