[Bug 40380] Add tp_dbus_properties_mixin_emit_properties_changed

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Fri Aug 26 17:37:15 CEST 2011


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

Jonny Lamb <jonny.lamb at collabora.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Status Whiteboard|                            |review+

--- Comment #5 from Jonny Lamb <jonny.lamb at collabora.co.uk> 2011-08-26 08:37:15 PDT ---
+ * @TP_DBUS_PROPERTIES_MIXIN_FLAG_EMITS_CHANGED: The property's new value is
+ * included in emissions of PropertiesChanged
+ * @TP_DBUS_PROPERTIES_MIXIN_FLAG_EMITS_INVALIDATED: The property is announced
+ * as invalidated, without its value, in emissions of PropertiesChanged

Perhaps include a comment in the docs mentioning that only one of these can be
used per property. The assertion later on the code isn't exactly obvious what
it's checking for either.

Looks fine otherwise.

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