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