empathy-tp-tube: remove initiator and type member variables as they are not used

[empathy.git] / libempathy / empathy-tube-handler.c

diff --git a/libempathy/empathy-tube-handler.c b/libempathy/empathy-tube-handler.c
index 0005a369d24840175e2a6b1af615968994cd7220..4fb82177da5bae4c77c8971ea668b5a1a48c4faf 100644 (file)
--- a/libempathy/empathy-tube-handler.c
+++ b/libempathy/empathy-tube-handler.c
@@ -27,14 +27,15 @@
 #include <telepathy-glib/connection.h>
 #include <telepathy-glib/channel.h>
 #include <telepathy-glib/interfaces.h>
+#include <telepathy-glib/util.h>
 
 #include <extensions/extensions.h>
 
-#include "empathy-debug.h"
 #include "empathy-tp-tube.h"
 #include "empathy-tube-handler.h"
 
-#define DEBUG_DOMAIN "TubeHandler"
+#define DEBUG_FLAG EMPATHY_DEBUG_OTHER
+#include "empathy-debug.h"
 
 static void empathy_tube_handler_iface_init (EmpSvcTubeHandlerClass *klass);
 
@@ -58,7 +59,6 @@ typedef struct
   gchar *channel;
   guint handle_type;
   guint handle;
-  guint id;
 } IdleData;
 
 static gboolean
@@ -70,7 +70,7 @@ tube_handler_handle_tube_idle_cb (gpointer data)
   EmpathyTpTube *tube;
   static TpDBusDaemon *daemon = NULL;
 
-  empathy_debug (DEBUG_DOMAIN, "New tube to be handled id=%d", idle_data->id);
+  DEBUG ("New tube to be handled");
 
   if (!daemon)
     daemon = tp_dbus_daemon_new (tp_get_bus ());
@@ -82,7 +82,7 @@ tube_handler_handle_tube_idle_cb (gpointer data)
       idle_data->handle, NULL);
   tp_channel_run_until_ready (channel, NULL, NULL);
 
-  tube = empathy_tp_tube_new (channel, idle_data->id);
+  tube = empathy_tp_tube_new (channel);
   g_signal_emit (idle_data->thandler, signals[NEW_TUBE], 0, tube);
 
   g_object_unref (tube);
@@ -103,7 +103,6 @@ tube_handler_handle_tube (EmpSvcTubeHandler *self,
                           const gchar *channel,
                           guint handle_type,
                           guint handle,
-                          guint id,
                           DBusGMethodInvocation *context)
 {
   EmpathyTubeHandler *thandler = EMPATHY_TUBE_HANDLER (self);
@@ -116,7 +115,6 @@ tube_handler_handle_tube (EmpSvcTubeHandler *self,
   data->channel = g_strdup (channel);
   data->handle_type = handle_type;
   data->handle = handle;
-  data->id = id;
 
   g_idle_add_full (G_PRIORITY_HIGH, tube_handler_handle_tube_idle_cb,
       data, NULL);
@@ -146,14 +144,21 @@ empathy_tube_handler_init (EmpathyTubeHandler *thandler)
 }
 
 EmpathyTubeHandler *
-empathy_tube_handler_new (const gchar *bus_name,
-                          const gchar *object_path)
+empathy_tube_handler_new (TpTubeType type, const gchar *service)
 {
-  EmpathyTubeHandler *thandler;
+  EmpathyTubeHandler *thandler = NULL;
   DBusGProxy *proxy;
   guint result;
+  gchar *bus_name;
+  gchar *object_path;
   GError *error = NULL;
 
+  g_return_val_if_fail (type < NUM_TP_TUBE_TYPES, NULL);
+  g_return_val_if_fail (service != NULL, NULL);
+
+  bus_name = empathy_tube_handler_build_bus_name (type, service);
+  object_path = empathy_tube_handler_build_object_path (type, service);
+
   proxy = dbus_g_proxy_new_for_name (tp_get_bus (), DBUS_SERVICE_DBUS,
       DBUS_PATH_DBUS, DBUS_INTERFACE_DBUS);
 
@@ -161,18 +166,61 @@ empathy_tube_handler_new (const gchar *bus_name,
       G_TYPE_STRING, bus_name, G_TYPE_UINT, DBUS_NAME_FLAG_DO_NOT_QUEUE,
       G_TYPE_INVALID, G_TYPE_UINT, &result, G_TYPE_INVALID))
     {
-      empathy_debug (DEBUG_DOMAIN, "Failed to request name: %s",
+      DEBUG ("Failed to request name: %s",
           error ? error->message : "No error given");
       g_clear_error (&error);
-      return NULL;
+      goto OUT;
     }
 
-  g_object_unref (proxy);
-
+  DEBUG ("Creating tube handler %s", bus_name);
   thandler = g_object_new (EMPATHY_TYPE_TUBE_HANDLER, NULL);
   dbus_g_connection_register_g_object (tp_get_bus (), object_path,
       G_OBJECT (thandler));
 
+OUT:
+  g_object_unref (proxy);
+  g_free (bus_name);
+  g_free (object_path);
+
   return thandler;
 }
 
+gchar *
+empathy_tube_handler_build_bus_name (TpTubeType type,
+  const gchar *service)
+{
+  gchar *str = NULL;
+  const gchar *prefix = NULL;
+
+  g_return_val_if_fail (type < NUM_TP_TUBE_TYPES, NULL);
+  g_return_val_if_fail (service != NULL, NULL);
+
+  if (type == TP_TUBE_TYPE_DBUS)
+    prefix = "org.gnome.Empathy.DTubeHandler.";
+  else if (type == TP_TUBE_TYPE_STREAM)
+    prefix = "org.gnome.Empathy.StreamTubeHandler.";
+  else
+    g_return_val_if_reached (NULL);
+
+  str = g_strconcat (prefix, service, NULL);
+
+  return str;
+}
+
+gchar *
+empathy_tube_handler_build_object_path (TpTubeType type,
+  const gchar *service)
+{
+  gchar *bus_name;
+  gchar *str;
+
+  g_return_val_if_fail (type < NUM_TP_TUBE_TYPES, NULL);
+  g_return_val_if_fail (service != NULL, NULL);
+
+  bus_name = empathy_tube_handler_build_bus_name (type, service);
+  str = g_strdelimit (g_strdup_printf ("/%s", bus_name), ".", '/');
+  g_free (bus_name);
+
+  return str;
+}
+