I don't like McdDBusProp
Submitted by Will Thompson @wjt
Assigned to Telepathy bugs list
Description
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.
Version: git master