[rhythmbox] visualizer: remove dbus interface
- From: Jonathan Matthew <jmatthew src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [rhythmbox] visualizer: remove dbus interface
- Date: Sun, 6 Mar 2011 12:17:04 +0000 (UTC)
commit 70fb9685eec77b733d85fff4f7342665432cf74c
Author: Jonathan Matthew <jonathan d14n org>
Date: Sun Mar 6 22:14:32 2011 +1000
visualizer: remove dbus interface
I'm pretty sure no one has ever used it.
plugins/visualizer/Makefile.am | 10 -----
plugins/visualizer/rb-visualizer-plugin.c | 61 -----------------------------
plugins/visualizer/rb-visualizer.xml | 17 --------
3 files changed, 0 insertions(+), 88 deletions(-)
---
diff --git a/plugins/visualizer/Makefile.am b/plugins/visualizer/Makefile.am
index 8376e73..5ded449 100644
--- a/plugins/visualizer/Makefile.am
+++ b/plugins/visualizer/Makefile.am
@@ -14,7 +14,6 @@ libvisualizer_la_LIBTOOLFLAGS = --tag=disable-static
libvisualizer_la_LIBADD = \
$(top_builddir)/shell/librhythmbox-core.la \
-lgstinterfaces-0.10 \
- $(DBUS_LIBS) \
$(RHYTHMBOX_LIBS)
INCLUDES = \
@@ -39,7 +38,6 @@ INCLUDES = \
-DDATADIR=\""$(datadir)"\" \
-DPLUGIN_SRC_DIR=\""$(ROOT_UNINSTALLED_DIR)/plugins/visualizer"\" \
$(RHYTHMBOX_CFLAGS) \
- $(DBUS_CFLAGS) \
-D_XOPEN_SOURCE -D_BSD_SOURCE
gtkbuilderdir = $(plugindir)
@@ -82,11 +80,3 @@ DISTCLEANFILES = \
$(BUILT_SOURCES) \
$(NULL)
-rb-visualizer-glue.h: rb-visualizer.xml Makefile
- $(LIBTOOL) --mode=execute $(DBUS_GLIB_BIN)/dbus-binding-tool --prefix=rb_visualizer --mode=glib-server --output=$@ $<
-
-BUILT_SOURCES += rb-visualizer-glue.h
-EXTRA_DIST += rb-visualizer.xml
-
-libvisualizer_la_LIBADD += $(DBUS_LIBS)
-
diff --git a/plugins/visualizer/rb-visualizer-plugin.c b/plugins/visualizer/rb-visualizer-plugin.c
index c21d807..12760a7 100644
--- a/plugins/visualizer/rb-visualizer-plugin.c
+++ b/plugins/visualizer/rb-visualizer-plugin.c
@@ -78,8 +78,6 @@
#define VISUALIZATION_ICON_NAME "visualization"
-#include <dbus/dbus-glib.h>
-
#include "rb-vis-widget.h"
/* preferences */
@@ -103,8 +101,6 @@
#define HIDE_CONTROLS_TIMEOUT 5 * 1000
-#define VISUALIZER_DBUS_PATH "/org/gnome/Rhythmbox/Visualizer"
-
/* playbin2 flag(s) */
#define PLAYBIN2_FLAG_VIS 0x08
@@ -195,10 +191,6 @@ typedef struct
GtkWidget *song_info_label;
GtkWidget *play_button;
gboolean syncing_play;
-
- gboolean dbus_interface_registered;
- gboolean plugin_enabled;
-
} RBVisualizerPlugin;
typedef struct
@@ -219,11 +211,6 @@ static void enable_visualization (RBVisualizerPlugin *pi);
static gboolean disable_visualization (RBVisualizerPlugin *pi, gboolean set_action);
static void update_window (RBVisualizerPlugin *plugin, VisualizerMode mode, int screen, int monitor);
-gboolean rb_visualizer_start_remote (RBVisualizerPlugin *plugin, unsigned long window_id, GError **error);
-gboolean rb_visualizer_stop_remote (RBVisualizerPlugin *plugin, GError **error);
-
-#include "rb-visualizer-glue.h"
-
static GtkToggleActionEntry rb_visualizer_plugin_toggle_actions [] =
{
{ "ToggleVisualizer", VISUALIZATION_ICON_NAME, N_("_Visualization"), NULL,
@@ -1800,23 +1787,6 @@ impl_activate (RBPlugin *plugin,
"window-title-changed",
G_CALLBACK (rb_visualizer_plugin_window_title_change_cb),
pi, 0);
-
- /* add dbus interface */
- if (pi->dbus_interface_registered == FALSE) {
- DBusGConnection *conn;
- GError *error = NULL;
-
- conn = dbus_g_bus_get (DBUS_BUS_SESSION, &error);
- if (conn != NULL) {
- dbus_g_object_type_install_info (RB_TYPE_VISUALIZER_PLUGIN,
- &dbus_glib_rb_visualizer_object_info);
- dbus_g_connection_register_g_object (conn,
- VISUALIZER_DBUS_PATH,
- G_OBJECT (plugin));
- pi->dbus_interface_registered = TRUE;
- }
- }
- pi->plugin_enabled = TRUE;
}
static void
@@ -1841,10 +1811,6 @@ impl_deactivate (RBPlugin *plugin,
pi->action_group = NULL;
}
- /* can't remove the dbus interface. it only goes away when the
- * plugin object does, which is when the process exits.
- */
-
g_object_unref (uim);
/* disconnect signal handlers */
@@ -1887,8 +1853,6 @@ impl_deactivate (RBPlugin *plugin,
if (pi->shell != NULL) {
pi->shell = NULL;
}
-
- pi->plugin_enabled = FALSE;
}
/* play controls (prev|play|next + song info (+ maybe seek bar) */
@@ -2370,28 +2334,3 @@ rb_visualizer_plugin_class_init (RBVisualizerPluginClass *klass)
plugin_class->activate = impl_activate;
plugin_class->deactivate = impl_deactivate;
}
-
-gboolean
-rb_visualizer_start_remote (RBVisualizerPlugin *plugin, unsigned long window_id, GError **error)
-{
- /* don't do anything is plugin is disabled */
- if (plugin->plugin_enabled == FALSE)
- return TRUE;
-
- /* this might not work properly - might need a new visualizer mode */
- plugin->remote_window = window_id;
- update_window (plugin, DESKTOP_WINDOW, -1, -1);
- return TRUE;
-}
-
-gboolean
-rb_visualizer_stop_remote (RBVisualizerPlugin *plugin, GError **error)
-{
- /* don't do anything is plugin is disabled */
- if (plugin->plugin_enabled == FALSE)
- return TRUE;
-
- plugin->remote_window = 0;
- update_window (plugin, EMBEDDED, -1, -1);
- return TRUE;
-}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]