Review Request 127189: ktp-contact-list offline-on-close fix.

James Smith smithjd15 at gmail.com
Mon Mar 7 07:10:51 UTC 2016


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

(Updated March 7, 2016, 7:10 a.m.)


Review request for Telepathy.


Repository: ktp-contact-list


Description
-------

-Remove Presence Applet cruft.
-Remove goOffline(), call setPresence() directly.

The check for qApp->closingDown() doesn't appear required, and removing it is needed for the global presence to change offline on close.

Applies to #123487, which already has GlobalPresence for offline-on-close as a requirement for multiple account presence setting.


Diffs (updated)
-----

  main-widget.h 43334ef8442217154f0df198ce601c9b996914d9 
  main-widget.cpp e0982594ac4c05b84db5772722b98b733410752b 

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


Testing
-------

Compile, run.


Thanks,

James Smith

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20160307/a7fc0ced/attachment.html>


More information about the KDE-Telepathy mailing list