[Bug 48230] New: [next] tp_am_set_default() should keep a weak ref instead of strong ref

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Tue Apr 3 12:01:46 CEST 2012


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

             Bug #: 48230
           Summary: [next] tp_am_set_default() should keep a weak ref
                    instead of strong ref
    Classification: Unclassified
           Product: Telepathy
           Version: unspecified
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: medium
         Component: tp-glib
        AssignedTo: telepathy-bugs at lists.freedesktop.org
        ReportedBy: xclaesse at gmail.com
         QAContact: telepathy-bugs at lists.freedesktop.org


tp_account_manager_set_default() adds an internal ref to @manager that can't be
be dropped. This can be annoying to debug leaks. Usually we only keep weak ref
to singletons, like tp_account_manager_dup() does.

Note: from a quick look, empathy actually depend on this behaviour since
empathy_init() does not keep the initial ref on its AM. So when/if fixing this
bug, we need to make sure to fix that in empathy too.

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



More information about the telepathy-bugs mailing list