[next] telepathy-glib: TpPresenceMixin: deprecate optional arguments other than "message"

Simon McVittie smcv at kemper.freedesktop.org
Mon Nov 11 12:14:03 PST 2013


Module: telepathy-glib
Branch: next
Commit: ccd7aed489d5e3263f0fa4c2f0af4141057aa7dd
URL:    http://cgit.freedesktop.org/telepathy/telepathy-glib/commit/?id=ccd7aed489d5e3263f0fa4c2f0af4141057aa7dd

Author: Simon McVittie <simon.mcvittie at collabora.co.uk>
Date:   Mon Oct 28 14:51:55 2013 +0000

TpPresenceMixin: deprecate optional arguments other than "message"

Our D-Bus API no longer supports anything else.

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes at collabora.co.uk>

---

 telepathy-glib/presence-mixin.c |   16 ++++++++++++++--
 1 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/telepathy-glib/presence-mixin.c b/telepathy-glib/presence-mixin.c
index e954b47..141dda8 100644
--- a/telepathy-glib/presence-mixin.c
+++ b/telepathy-glib/presence-mixin.c
@@ -131,7 +131,10 @@
  * @optional_arguments: An array of #TpPresenceStatusOptionalArgumentSpec
  *  structures representing the optional arguments for this status, terminated
  *  by a NULL name. If there are no optional arguments for a status, this can
- *  be NULL.
+ *  be NULL. In modern Telepathy connection managers, the only optional
+ *  argument should be a string (type "s") named "message" on statuses
+ *  that have an optional human-readable message. All other optional arguments
+ *  are deprecated.
  *
  * Structure specifying a supported presence status.
  *
@@ -153,9 +156,13 @@
  * In addition to the fields documented here, there are two gpointer fields
  * which must currently be %NULL. A meaning may be defined for these in a
  * future version of telepathy-glib.
+ *
+ * In modern Telepathy connection managers, the only optional
+ * argument should be a %G_TYPE_STRING named "message", on statuses
+ * that have an optional human-readable message. All other optional arguments
+ * are deprecated.
  */
 
-
 /**
  * TpPresenceMixinStatusAvailableFunc:
  * @obj: An instance of a #TpBaseConnection subclass implementing the presence
@@ -333,6 +340,11 @@ deep_copy_hashtable (GHashTable *hash_table)
  * Construct a presence status structure. You should free the returned
  * structure with #tp_presence_status_free.
  *
+ * In modern Telepathy connection managers, the only optional
+ * argument should be a %G_TYPE_STRING named "message", on statuses
+ * that have an optional human-readable message. All other optional arguments
+ * are deprecated.
+ *
  * Returns: A pointer to the newly allocated presence status structure.
  */
 TpPresenceStatus *



More information about the telepathy-commits mailing list