[Bug 78376] New: remove tp_dbus_properties_mixin_class_init(), use "invisible mixin"

[bugzilla-daemon at freedesktop.org]

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

          Priority: medium
            Bug ID: 78376
          Assignee: telepathy-bugs at lists.freedesktop.org
            Blocks: 68661
           Summary: remove tp_dbus_properties_mixin_class_init(), use
                    "invisible mixin"
        QA Contact: telepathy-bugs at lists.freedesktop.org
          Severity: normal
    Classification: Unclassified
                OS: All
          Reporter: simon.mcvittie at collabora.co.uk
          Hardware: Other
            Status: NEW
           Version: git master
         Component: tp-glib
        Depends on: 77189
           Product: Telepathy

Xavier writes
> We could use the "invisible mixin" in all CMs to kill
> tp_dbus_properties_mixin_class_init() and all code paths where offset!=0? I'm 
> happier now that it is in -dbus because it should really be considered 
> deprecated and we should try to remove it completely from all CMs on the long 
> term.

to which I replied
> I think I might actually have eradicated offset != 0 while updating the CMs
> (it seemed cleaner to use the invisible mixin than to include
> telepathy-glib-dbus.h in their headers) but this is way off the critical
> path for a clean and future-proof libtelepathy-glib-1.so API.

TpDBusPropertiesMixin is in libtelepathy-glib-1-dbus.so now, so it isn't
critical-path, but would be nice.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.