[Bug 31031] Add (optional) support for Nokia's heartbeat service

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Fri Oct 22 20:11:10 CEST 2010


https://bugs.freedesktop.org/show_bug.cgi?id=31031

Vivek Dasmohapatra <vivek at collabora.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Status Whiteboard|                            |review-

--- Comment #2 from Vivek Dasmohapatra <vivek at collabora.co.uk> 2010-10-22 11:11:07 PDT ---
Nothing major: 1 trivium and one question:

Trivium:

where DEBUG output is to tell us there's been a wakeup or a ping 
has gone out or the ping timeout has changed, it might be worth 
logging the (high-ish res)time at which said event occurred

Question:

self->next_wakeup.tv_sec += (max_interval - self->max_interval);
can go -ve, I _think_ this means we just wake up straightaway,
I assume that's Ok?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.



More information about the telepathy-bugs mailing list