Review Request 127699: Set the xscreensaver plugin as valid only if the dbus interface org.freedesktop.ScreenSaver is present

Paulo Lieuthier paulolieuthier at gmail.com
Mon Apr 25 14:52:44 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127699/
-----------------------------------------------------------

(Updated April 25, 2016, 11:52 a.m.)


Status
------

This change has been discarded.


Review request for KDE Frameworks and Martin Gräßlin.


Repository: kidletime


Description
-------

If the dbus interface org.freedesktop.ScreenSaver is not present, it makes no sense to use the plugin.


Diffs
-----

  src/plugins/xscreensaver/xscreensaverbasedpoller.h 05b9a23 
  src/plugins/xscreensaver/xscreensaverbasedpoller.cpp f56396c 
  src/widgetbasedpoller.h 68fc7dc 

Diff: https://git.reviewboard.kde.org/r/127699/diff/


Testing
-------

Tested minimal application using KIdleTime using the xscreensaver plugin.


Thanks,

Paulo Lieuthier

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160425/7e32d8a4/attachment.html>


More information about the Kde-frameworks-devel mailing list