[Kde-hardware-devel] Review Request 111699: Fixed "no such signal" message in UPowerManager at startup

[Albert Vaca Cintora]

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

(Updated July 28, 2013, 5:07 p.m.)


Status
------

This change has been discarded.


Review request for Solid and Àlex Fiestas.


Description
-------

The following message was shown when UPowerManager was instantiated (e.g.: when kded4 starts):

Object::connect: No such signal QDBusAbstractInterface::DeviceAdded(QString) in /home/vaka/kde4/src/kdelibs/solid/solid/backends/upower/upowermanager.cpp:67
Object::connect: No such signal QDBusAbstractInterface::DeviceRemoved(QString) in /home/vaka/kde4/src/kdelibs/solid/solid/backends/upower/upowermanager.cpp:69

This patch fixes to connection to the mentioned DBus signal (the normal connect construction that was used doesn't work for dbus signals) so this message doesn't appear anymore, and probably fixes the underlying functionality that required this signal to be connected.


Diffs
-----

  solid/solid/backends/upower/upowermanager.cpp bae234b 

Diff: http://git.reviewboard.kde.org/r/111699/diff/


Testing
-------

Manual testing.


Thanks,

Albert Vaca Cintora

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20130728/3a872503/attachment-0001.html>