Bug 54762 - Stop using tp_handle_set_qdata()
Summary: Stop using tp_handle_set_qdata()
Status: NEW
Alias: None
Product: Telepathy
Classification: Unclassified
Component: gabble (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-09-11 09:01 UTC by Xavier Claessens
Modified: 2013-12-04 03:15 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Xavier Claessens 2012-09-11 09:01:36 UTC
Handles are immortal so this leaks the data until the connection disconnects.

<smcv> I think we're going to need some sort of heuristic analogous to what would previously have kept handles reffed. keep = (mentioned by contact list || target of a channel || in a chatroom)
<smcv> might be worth introducing a GabbleContact object similar to Salut's SalutContact and caching the alias/vCard there?

This should happen in gabble master, tp_handle_set/get_qdata() has been deprecated in tp-glib 0.19.9 and will be removed in -next.
Comment 1 Xavier Claessens 2013-12-04 03:15:04 UTC
AFAIK this has been already fixed, right?


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.