[Bug 32416] New: I don't like McdDBusProp

[bugzilla-daemon at freedesktop.org]

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

           Summary: I don't like McdDBusProp
           Product: Telepathy
           Version: git master
          Platform: Other
               URL: http://git.collabora.co.uk/?p=user/wjt/telepathy-missi
                    on-control-wjt.git;a=commitdiff;h=refs/heads/get-rid-o
                    f-McdDBusProp
        OS/Version: All
            Status: NEW
          Severity: minor
          Priority: low
         Component: mission-control
        AssignedTo: telepathy-bugs at lists.freedesktop.org
        ReportedBy: will.thompson at collabora.co.uk
         QAContact: telepathy-bugs at lists.freedesktop.org


I don't like McdDBusProp. It seems stupid that we have this one implementation
of properties—TpDBusPropertiesMixin—but one of the main Telepathy components
doesn't use it.

So I made a stab at getting rid of it, starting by porting McdAccount to use
TpDBusPropertiesMixin. The attached branch is *almost* there … except that it
breaks creating accounts. McdAccountManager calls mcd_dbusprop_set_property()
on McdAccount to set all the initial properties, and I can't see any way to do
the equivalent using TpDBusPropertiesMixin (short of rootling around in its
internal qdata and generally being a bad person). So I've given up for now.

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