Review Request 125028: Monitor for the clock changes from the kernel

David Edmundson david at davidedmundson.co.uk
Fri Sep 11 15:54:56 UTC 2015


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

(Updated Sept. 11, 2015, 3:54 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Changes
-------

Submitted with commit bbfffc2d65ab53acd41ee056fdd2a6ea9c2f6a08 by David Edmundson to branch master.


Repository: plasma-workspace


Description
-------

At the moment we update the clock when we resume from suspend or from
the time KCM.

However the signal from the clock KCM isn't very good as it signals just
after it requests an NTP update or clock change, not necessarily when
that finishes.

Also we miss any external NTP updates which might occur at other times
throughout the day, such as a big change when we connect to the network

This method also avoids needing solid to tell us when we're resuming
from suspend, which is deprecated API and also seems slightly slower.

BUG: 344870


Diffs
-----

  dataengines/time/timeengine.cpp 2fd9792209ff5e78bd3dee1ed938eb0b1173de8d 

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


Testing
-------


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150911/62a3b916/attachment.html>


More information about the Plasma-devel mailing list