[Bug 23148] Telepathy 1.0: break D-Bus API to remove deprecated stuff (metabug)

[bugzilla-daemon at freedesktop.org]

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

--- Comment #21 from Simon McVittie <simon.mcvittie at collabora.co.uk> ---
(In reply to comment #19)
> (In reply to comment #8)
> > > > * rename Handle_Type to Target_Type and TargetHandleType to TargetType
> > > 
> > > Do we still want this?
> > 
> > Given that we've mostly eradicated handles from our high-level API, it seems
> > somewhat appealing to remove that bit of jargon too; OTOH, perhaps this is
> > one sed too many.
> 
> Shouldn't we remove Channel.TargetHandle as well then?

We still need handles *and* IDs for as long as our general-purpose APIs (mostly
meaning the ones called by TpContact) primarily take handles rather than IDs as
their parameters. For instance, RequestContactInfo() takes a handle.

All I'm trying to do right now is to change the conceptual model from:

    A channel has a *target handle*, which (like all handles) has a
    *handle type*. It also has a *target identifier* which is the string
    form of the handle

to:

    A channel has a *target*, represented as a pair
    (target type, identifier) or equivalently (target type, handle)

which makes it a D-Bus-level implementation detail whether the handle or the
identifier is more useful.

I don't think it's worthwhile to change all the D-Bus APIs from handle-based to
ID-based for Telepathy 1.0, but what I'm aiming for here is that if we want to,
we can make that change for (say) Telepathy 1.2 by breaking D-Bus API, without
having to also break the GLib- and Qt-level API.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.