From 3666a97436a0d17e482bb33f3c3a23f43576d05d Mon Sep 17 00:00:00 2001 From: Simon McVittie Date: Mon, 28 Oct 2013 14:51:55 +0000 Subject: [PATCH 08/15] TpPresenceMixin: deprecate optional arguments other than "message" Our D-Bus API no longer supports anything else. --- telepathy-glib/presence-mixin.c | 16 ++++++++++++++-- 1 file 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 * -- 1.8.4.rc3