[Bug 38036] TpConnection: add Location support

[bugzilla-daemon at freedesktop.org]

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

--- Comment #2 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2011-06-08 06:45:06 PDT ---
(In reply to comment #1)
> - I'm not sure how we want to represent location; maybe we want something
> better than a TpAsv?

It'd be good to follow tp-qt4's lead and return an opaque thing (GObject, or
possibly boxed struct), with convenience accessors for the well-known keys
(tp_location_get_latitude() etc.) plus a way to peek at the actual contents.
Migrating to GDBus or changing the D-Bus API would break the API of the latter
but not the former.

Complications for the convenience API:

* everything needs to be "nullable" ("not known" is always a possible answer)

* some properties only make sense in groups - e.g. perhaps have:

    /**
     * lat: (out): latitude in degrees north (negative if south of the
Equator),
     *    undefined if %FALSE is returned
     * lat: (out): longitude in degrees east (negative if west of Greenwich),
     *    undefined if %FALSE is returned
     * Returns: %TRUE if both @lat and @long are meaningful
     */
    gboolean tp_location_get_coordinates (TpLocation *, gdouble *lat,
        gdouble *long);

-- 
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.