Bug 29459 - ReferencedHandles doesn't need to be plural
Summary: ReferencedHandles doesn't need to be plural
Status: RESOLVED NOTABUG
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-qt (show other bugs)
Version: git master
Hardware: All All
: medium enhancement
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL:
Whiteboard:
Keywords:
Depends on: 23155
Blocks:
  Show dependency treegraph
 
Reported: 2010-08-09 09:05 UTC by Olli Salli
Modified: 2010-11-05 01:06 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Olli Salli 2010-08-09 09:05:05 UTC
Tp-Qt4 does application-level RAII-powered reference counting on handles, so that the application doesn't have to care about the D-Bus level semantics of HoldHandles/ReleaseHandles. In an initial frenzy of trying to enforce batching of the D-Bus level operations as much as possible, I designed the API as a container of handles. While it is extremely useful to store a single handle in isolation (eg. the handle of a single Contact), this can only be represented as a ReferencedHandles container with size one.

However, it turned out that trying to keep big bunches of ReferencedHandles together wasn't really very practical, and that implementing batch combining logic was sufficiently easy, so in the end, all handles are actually released one-by-one by ReferencedHandles, and the Connection takes care to form a efficient batch of them (actually all handles which lost their references in any container in a given Qt mainloop iteration). This obviously means the API could be ReferencedHandle instead, and plural operations could simply be QList<ReferencedHandle>.

This is obviously an API/ABI break, so we shouldn't consider doing it until 1.0. Some points to consider then, though:
 * ReferencedHandles currently also makes sure that all handles within are from the same actual Connection - should we preserve it as a thin means to this end alone?
 * Should handles actually be made immortal on the D-Bus level (bug #23155)
  - ... or should we <shock> actually just use normalized ID strings (but still with CM-level normalization) wherever we currently use handles (the TargetID, etc, additions from a few spec major versions ago are already half-way there anyway)
Comment 1 Olli Salli 2010-08-19 02:48:46 UTC
We're going to have an API/ABI break really soon now, so let's target this for inclusion too.
Comment 2 Olli Salli 2010-11-05 01:06:05 UTC
The handle concept is low-level enough to make this not much of a problem; we can always wrap handle APIs with overloads taking ContactPtrs and/or IDs to make using handles even more easier (because tp-qt4 does it for you!).


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.