Attachment 36205 Details for Bug 28312 – Adds TpContact::presence-changed signal

[patch] Adds TpContact::presence-changed signal

0001-Adds-TpContact-presence-changed-signal.patch (text/plain), 3.54 KB, created by Morten Mjelva on 2010-06-10 10:50:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Morten Mjelva

Created: 2010-06-10 10:50:04 UTC

Size: 3.54 KB

patch

obsolete

>From ba8e1f1e153ade73d870ad90944516af4f9f7b33 Mon Sep 17 00:00:00 2001
>From: mortenmj <morten.mjelva@gmail.com>
>Date: Thu, 10 Jun 2010 19:03:56 +0200
>Subject: [PATCH] Adds TpContact::presence-changed signal
>
>This adds a signal to TpContact that is issued whenever the PresencesChanged
>signal on org.freedesktop.Telepathy.Connection.SimplePresence is issued for
>the contact in question. The signal holds the new presence information for the
>contact.
>---
> telepathy-glib/contact.c |   55 ++++++++++++++++++++++++++++++++++++++++------
> 1 files changed, 48 insertions(+), 7 deletions(-)
>
>diff --git a/telepathy-glib/contact.c b/telepathy-glib/contact.c
>index 78afe54..7826d0a 100644
>--- a/telepathy-glib/contact.c
>+++ b/telepathy-glib/contact.c
>@@ -32,6 +32,8 @@
> #include "telepathy-glib/connection-internal.h"
> #include "telepathy-glib/debug-internal.h"
> 
>+#include "telepathy-glib/_gen/signals-marshal.h"
>+
> /**
>  * SECTION:contact
>  * @title: TpContact
>@@ -135,6 +137,13 @@ enum {
>     N_PROPS
> };
> 
>+enum {
>+    SIGNAL_PRESENCE_CHANGED,
>+    N_SIGNALS
>+};
>+
>+static guint signals[N_SIGNALS] = {0};
>+
> /* The API allows for more than 32 features, but this implementation does
>  * not. We can easily expand this later. */
> typedef enum {
>@@ -858,6 +867,24 @@ tp_contact_class_init (TpContactClass *klass)
>       G_PARAM_READABLE | G_PARAM_STATIC_STRINGS);
>   g_object_class_install_property (object_class, PROP_CAPABILITIES,
>       param_spec);
>+ 
>+
>+  /**
>+   * TpContact::presence-changed:
>+   * @contact: A #TpContact
>+   * @presence: The contact's presence type 
>+   * @status: Protocol-defined description of the presence type
>+   * @status_message: User-defined status message
>+   *
>+   * Emitted when PresenceChanged is received for this contact.
>+   */
>+  signals[SIGNAL_PRESENCE_CHANGED] = g_signal_new ("presence-changed",
>+      G_TYPE_FROM_CLASS (object_class),
>+      G_SIGNAL_RUN_LAST,
>+      0,
>+      NULL, NULL,
>+      _tp_marshal_VOID__UINT_STRING_STRING,
>+      G_TYPE_NONE, 3, G_TYPE_UINT, G_TYPE_STRING, G_TYPE_STRING);
> }
> 
> 
>@@ -1632,25 +1659,39 @@ contacts_get_aliases (ContactsContext *c)
> }
> 
> 
>+// Presence properties might have changed. 
>+// Set new values, and emit the TpContact::presence-changed signal.
> static void
> contact_maybe_set_simple_presence (TpContact *contact,
>                                    GValueArray *presence)
> {
>+  guint type;
>+  const gchar *status;
>+  const gchar *message;
>+
>   if (contact == NULL || presence == NULL)
>     return;
> 
>   contact->priv->has_features |= CONTACT_FEATURE_FLAG_PRESENCE;
>-  contact->priv->presence_type = g_value_get_uint (presence->values + 0);
>-  g_free (contact->priv->presence_status);
>-  contact->priv->presence_status = g_value_dup_string (
>-      presence->values + 1);
>-  g_free (contact->priv->presence_message);
>-  contact->priv->presence_message = g_value_dup_string (
>-      presence->values + 2);
> 
>+  tp_value_array_unpack (presence, presence->n_values, &type, &status,
>+      &message);
>+
>+  contact->priv->presence_type = type;
>   g_object_notify ((GObject *) contact, "presence-type");
>+
>+  g_free (contact->priv->presence_status);
>+  contact->priv->presence_status = g_strdup (status);
>   g_object_notify ((GObject *) contact, "presence-status");
>+
>+  g_free (contact->priv->presence_message);
>+  contact->priv->presence_message = g_strdup (message);
>   g_object_notify ((GObject *) contact, "presence-message");
>+
>+  g_signal_emit (contact, signals[SIGNAL_PRESENCE_CHANGED], 0,
>+      contact->priv->presence_type,
>+      contact->priv->presence_status,
>+      contact->priv->presence_message);
> }
> 
> static void
>-- 
>1.7.0.4
>

Actions: View | Splinter Review

Attachments on bug 28312: 36205 | 36227 | 36261

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.