Review Request 122590: Guard kglobalaccel against QApplication
Kai Uwe Broulik
kde at privat.broulik.de
Mon Feb 16 11:56:48 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122590/
-----------------------------------------------------------
(Updated Feb. 16, 2015, 11:56 vorm.)
Review request for KDE Frameworks, David Edmundson and Martin Gräßlin.
Changes
-------
Use QScopedPointer
Repository: kglobalaccel
Description
-------
KGlobalAccelPrivate uses a QWidget for some magic witchcraft which blows up in case there is no QApplication (kscreenlocker_greet). This guards against it.
Diffs (updated)
-----
src/kglobalaccel.cpp df85547
src/kglobalaccel_p.h b1528dc
Diff: https://git.reviewboard.kde.org/r/122590/diff/
Testing
-------
screenlocker no longer blows up when adding the mpris dataengine which does kglobalaccel stuff but I have no idea whether this is the right approach and what the widget is actually for.
Thanks,
Kai Uwe Broulik
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150216/e3bfee76/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list