Re: Adding a LastSeen property to accesspoints and ScanDone signal



On Thu, Apr 2, 2015 at 11:49 AM, Dan Williams <dcbw redhat com> wrote:
[...]

Patch #1 needs some fixups, namely:

1) fix the D-Bus property documentation, and mention that the value is
absolute in CLOCK_BOOTTIME seconds, so CLOCK_BOOTTIME seconds -
last_seen = seconds ago the AP was last seen

2) make the property a u32 (it'll never be negative)

3) same thing for libnm-glib, a u32, and update the code documentation
to talk about CLOCK_BOOTTIME etc

4) for nm-wifi-ap.c, keep the current 'gint' stuff for
nm_ap_set_last_seen(), but in get_property() convert last_seen into
BOOTTIME units with nm_utils_monotonic_timestamp_as_boottime() from
Thomas' patch so that D-Bus clients see BOOTTIME, and add a code comment
about this in get_property()

Sound OK?

Yup. Will get you updated patches today.
 

--

Mathieu Trudel-Lapierre <mathieu trudel-lapierre canonical com>
Freenode: cyphermox, Jabber: mathieu tl gmail com
4096R/EE018C93 1967 8F7D 03A1 8F38 732E  FF82 C126 33E1 EE01 8C93


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]