[glib] Revert "GDBusActionGroup: add static platform registration"



commit e5ed11bcf807b3e2c0a36c684e9b794ec253cd72
Author: Ryan Lortie <desrt desrt ca>
Date:   Fri Dec 16 21:25:57 2011 -0500

    Revert "GDBusActionGroup: add static platform registration"
    
    This reverts commit fcc9902e98a6a6568a1150441c8680fd4615d437.

 gio/gactiongroupexporter.c |   10 ----------
 gio/gdbusactiongroup.c     |   32 ++------------------------------
 gio/gdbusactiongroup.h     |   14 +++-----------
 gio/gio.symbols            |    1 -
 4 files changed, 5 insertions(+), 52 deletions(-)
---
diff --git a/gio/gactiongroupexporter.c b/gio/gactiongroupexporter.c
index 8f3fccb..a399fff 100644
--- a/gio/gactiongroupexporter.c
+++ b/gio/gactiongroupexporter.c
@@ -27,14 +27,10 @@
 #include "gdbusmethodinvocation.h"
 #include "gdbusintrospection.h"
 #include "gdbusconnection.h"
-#include "gdbusactiongroup.h"
 #include "gactiongroup.h"
 #include "gapplication.h"
 #include "gdbuserror.h"
 
-extern GDBusActionGroupEmitHookFunc g_dbus_action_group_before_emit_hook;
-extern GDBusActionGroupEmitHookFunc g_dbus_action_group_after_emit_hook;
-
 /**
  * SECTION:gactiongroupexporter
  * @title: GActionGroup exporter
@@ -453,9 +449,6 @@ g_action_group_exporter_pre_emit (GActionGroupExporter *exporter,
   if (G_IS_APPLICATION (exporter->action_group))
     G_APPLICATION_GET_CLASS (exporter->action_group)
       ->before_emit (G_APPLICATION (exporter->action_group), platform_data);
-
-  else if (g_dbus_action_group_before_emit_hook != NULL)
-    (* g_dbus_action_group_before_emit_hook) (exporter->action_group, platform_data);
 }
 
 static void
@@ -465,9 +458,6 @@ g_action_group_exporter_post_emit (GActionGroupExporter *exporter,
   if (G_IS_APPLICATION (exporter->action_group))
     G_APPLICATION_GET_CLASS (exporter->action_group)
       ->after_emit (G_APPLICATION (exporter->action_group), platform_data);
-
-  else if (g_dbus_action_group_after_emit_hook != NULL)
-    (* g_dbus_action_group_after_emit_hook) (exporter->action_group, platform_data);
 }
 
 static void
diff --git a/gio/gdbusactiongroup.c b/gio/gdbusactiongroup.c
index c3353c1..7c7b38e 100644
--- a/gio/gdbusactiongroup.c
+++ b/gio/gdbusactiongroup.c
@@ -38,10 +38,6 @@
  * that is exported over D-Bus with g_dbus_connection_export_action_group().
  */
 
-G_GNUC_INTERNAL GDBusActionGroupSendHookFunc g_dbus_action_group_send_hook;
-G_GNUC_INTERNAL GDBusActionGroupEmitHookFunc g_dbus_action_group_before_emit_hook;
-G_GNUC_INTERNAL GDBusActionGroupEmitHookFunc g_dbus_action_group_after_emit_hook;
-
 struct _GDBusActionGroup
 {
   GObject parent_instance;
@@ -372,15 +368,10 @@ g_dbus_action_group_change_state (GActionGroup *g_group,
                                   GVariant     *value)
 {
   GDBusActionGroup *group = G_DBUS_ACTION_GROUP (g_group);
-  GVariantBuilder platform_data_builder;
-
-  g_variant_builder_init (&platform_data_builder, G_VARIANT_TYPE_VARDICT);
-  if (g_dbus_action_group_send_hook != NULL)
-    (* g_dbus_action_group_send_hook) (group, &platform_data_builder);
 
   /* Don't bother with the checks.  The other side will do it again. */
   g_dbus_connection_call (group->connection, group->bus_name, group->object_path, "org.gtk.Actions", "SetState",
-                          g_variant_new ("(sva{sv})", action_name, value, &platform_data_builder),
+                          g_variant_new ("(sva{sv})", action_name, value, NULL),
                           NULL, G_DBUS_CALL_FLAGS_NONE, -1, NULL, NULL, NULL);
 }
 
@@ -390,7 +381,6 @@ g_dbus_action_group_activate (GActionGroup *g_group,
                               GVariant     *parameter)
 {
   GDBusActionGroup *group = G_DBUS_ACTION_GROUP (g_group);
-  GVariantBuilder platform_data_builder;
   GVariantBuilder builder;
 
   g_variant_builder_init (&builder, G_VARIANT_TYPE ("av"));
@@ -398,12 +388,8 @@ g_dbus_action_group_activate (GActionGroup *g_group,
   if (parameter)
     g_variant_builder_add (&builder, "v", parameter);
 
-  g_variant_builder_init (&platform_data_builder, G_VARIANT_TYPE_VARDICT);
-  if (g_dbus_action_group_send_hook != NULL)
-    (* g_dbus_action_group_send_hook) (group, &platform_data_builder);
-
   g_dbus_connection_call (group->connection, group->bus_name, group->object_path, "org.gtk.Actions", "Activate",
-                          g_variant_new ("(sava{sv})", action_name, &builder, &platform_data_builder),
+                          g_variant_new ("(sava{sv})", action_name, &builder, NULL),
                           NULL, G_DBUS_CALL_FLAGS_NONE, -1, NULL, NULL, NULL);
 }
 
@@ -521,17 +507,3 @@ g_dbus_action_group_sync (GDBusActionGroup  *group,
 
   return reply != NULL;
 }
-
-void
-g_dbus_action_group_register_platform (GDBusActionGroupSendHookFunc send,
-                                       GDBusActionGroupEmitHookFunc before,
-                                       GDBusActionGroupEmitHookFunc after)
-{
-  g_return_if_fail (g_dbus_action_group_send_hook == NULL &&
-                    g_dbus_action_group_before_emit_hook == NULL &&
-                    g_dbus_action_group_after_emit_hook == NULL);
-
-  g_dbus_action_group_send_hook = send;
-  g_dbus_action_group_before_emit_hook = before;
-  g_dbus_action_group_after_emit_hook = after;
-}
diff --git a/gio/gdbusactiongroup.h b/gio/gdbusactiongroup.h
index d5b9b1d..12a8edf 100644
--- a/gio/gdbusactiongroup.h
+++ b/gio/gdbusactiongroup.h
@@ -45,17 +45,9 @@ G_BEGIN_DECLS
 
 GType                   g_dbus_action_group_get_type                  (void) G_GNUC_CONST;
 
-GDBusActionGroup *      g_dbus_action_group_get                       (GDBusConnection              *connection,
-                                                                       const gchar                  *bus_name,
-                                                                       const gchar                  *object_path);
-
-typedef void         (* GDBusActionGroupSendHookFunc)                 (GDBusActionGroup             *group,
-                                                                       GVariantBuilder              *builder);
-typedef void         (* GDBusActionGroupEmitHookFunc)                 (GActionGroup                 *group,
-                                                                       GVariant                     *platform_data);
-void                    g_dbus_action_group_register_platform         (GDBusActionGroupSendHookFunc  send_hook,
-                                                                       GDBusActionGroupEmitHookFunc  before_hook,
-                                                                       GDBusActionGroupEmitHookFunc  after_hook);
+GDBusActionGroup *      g_dbus_action_group_get                       (GDBusConnection        *connection,
+                                                                       const gchar            *bus_name,
+                                                                       const gchar            *object_path);
 
 G_END_DECLS
 
diff --git a/gio/gio.symbols b/gio/gio.symbols
index a2d9469..8fed849 100644
--- a/gio/gio.symbols
+++ b/gio/gio.symbols
@@ -1442,7 +1442,6 @@ g_simple_action_new_stateful
 g_simple_action_set_enabled
 g_simple_action_set_state
 g_dbus_action_group_get_type
-g_dbus_action_group_register_platform
 g_dbus_action_group_get
 g_dbus_menu_model_get_type
 g_dbus_menu_model_get



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]