Remove extra parameter from config() plugin callback - parent window is panel.
authorAndriy Grytsenko <andrej@rep.kiev.ua>
Fri, 15 Aug 2014 23:37:40 +0000 (02:37 +0300)
committerAndriy Grytsenko <andrej@rep.kiev.ua>
Sat, 16 Aug 2014 16:23:41 +0000 (19:23 +0300)
20 files changed:
src/configurator.c
src/plugin.c
src/plugin.h
src/plugins/batt/batt.c
src/plugins/dclock.c
src/plugins/deskno/deskno.c
src/plugins/dirmenu.c
src/plugins/indicator/indicator.c
src/plugins/kbled/kbled.c
src/plugins/launchtaskbar.c
src/plugins/menu.c
src/plugins/monitors/monitors.c
src/plugins/netstatus/netstatus.c
src/plugins/pager.c
src/plugins/space.c
src/plugins/thermal/thermal.c
src/plugins/volumealsa/volumealsa.c
src/plugins/wincmd.c
src/plugins/xkb/xkb-plugin.c
src/private.h

index 18c5ccf..046adfc 100644 (file)
@@ -824,7 +824,7 @@ void modify_plugin( GtkTreeView* view )
     {
         GtkWidget *dlg;
         LXPanel *panel = PLUGIN_PANEL(pl);
-        dlg = init->config(panel, pl, GTK_WINDOW(gtk_widget_get_toplevel(GTK_WIDGET(view))));
+        dlg = init->config(panel, pl);
         if (dlg)
             _panel_show_config_dialog(panel, pl, dlg);
     }
index 8b5286d..3837a2c 100644 (file)
@@ -41,6 +41,8 @@
 static void init_plugin_class_list(void);
 static void plugin_class_unref(PluginClass * pc);
 
+GQuark lxpanel_plugin_qinit;
+GQuark lxpanel_plugin_qconf;
 GQuark lxpanel_plugin_qdata;
 static GHashTable *_all_types = NULL;
 
@@ -61,7 +63,7 @@ static inline LXPanelPluginInit *_find_plugin(const char *name)
     return g_hash_table_lookup(_all_types, name);
 }
 
-static GtkWidget *_old_plugin_config(LXPanel *panel, GtkWidget *instance, GtkWindow *parent)
+static GtkWidget *_old_plugin_config(LXPanel *panel, GtkWidget *instance)
 {
     LXPanelPluginInit *init = PLUGIN_CLASS(instance);
     Plugin * plugin;
@@ -69,7 +71,7 @@ static GtkWidget *_old_plugin_config(LXPanel *panel, GtkWidget *instance, GtkWin
     g_return_val_if_fail(init != NULL && init->new_instance == NULL, NULL);
     plugin = lxpanel_plugin_get_data(instance);
     if (plugin->class->config)
-        plugin->class->config(plugin, parent);
+        plugin->class->config(plugin, GTK_WINDOW(panel));
     return NULL;
 }
 
@@ -435,7 +437,7 @@ void lxpanel_plugin_show_config_dialog(GtkWidget* plugin)
     if (dlg && g_object_get_data(G_OBJECT(dlg), "generic-config-plugin") == plugin)
         return; /* configuration dialog is already shown for this widget */
     g_return_if_fail(panel != NULL);
-    dlg = init->config(panel, plugin, GTK_WINDOW(panel));
+    dlg = init->config(panel, plugin);
     if (dlg)
         _panel_show_config_dialog(panel, plugin, dlg);
 }
index 860e45c..7c6ee86 100644 (file)
@@ -84,7 +84,7 @@ G_BEGIN_DECLS
  *
  * Callback @control is called when command was sent via the lxpanelctl.
  * The message will be sent to only one instance of plugin. Some messages
- * are handled by lxpanel: "STOP" will remove plugin from panel, "START"
+ * are handled by lxpanel: "DEL" will remove plugin from panel, "ADD"
  * will create new instance if there is no instance yet. (TODO)
  */
 typedef struct {
@@ -94,7 +94,7 @@ typedef struct {
     char *name;                 /* name to represent in lists */
     char *description;          /* tooltip text */
     GtkWidget *(*new_instance)(LXPanel *panel, config_setting_t *settings);
-    GtkWidget *(*config)(LXPanel *panel, GtkWidget *instance, GtkWindow *parent);
+    GtkWidget *(*config)(LXPanel *panel, GtkWidget *instance);
     void (*reconfigure)(LXPanel *panel, GtkWidget *instance);
     gboolean (*button_press_event)(GtkWidget *widget, GdkEventButton *event, LXPanel *panel);
     void (*show_system_menu)(GtkWidget *widget);
index ce4fb10..cb6c5dc 100644 (file)
@@ -635,7 +635,7 @@ static gboolean applyConfig(gpointer user_data)
 }
 
 
-static GtkWidget *config(LXPanel *panel, GtkWidget *p, GtkWindow *parent) {
+static GtkWidget *config(LXPanel *panel, GtkWidget *p) {
     lx_battery *b = lxpanel_plugin_get_data(p);
     return lxpanel_generic_config_dlg(_("Battery Monitor"),
             panel, applyConfig, p,
index 5ce6e98..ffc5656 100644 (file)
@@ -415,7 +415,7 @@ static gboolean dclock_apply_configuration(gpointer user_data)
 }
 
 /* Callback when the configuration dialog is to be shown. */
-static GtkWidget *dclock_configure(LXPanel *panel, GtkWidget *p, GtkWindow *parent)
+static GtkWidget *dclock_configure(LXPanel *panel, GtkWidget *p)
 {
     DClockPlugin * dc = lxpanel_plugin_get_data(p);
     return lxpanel_generic_config_dlg(_("Digital Clock"), panel,
index af21ca7..765d064 100644 (file)
@@ -168,7 +168,7 @@ static gboolean deskno_apply_configuration(gpointer user_data)
 }
 
 /* Callback when the configuration dialog is to be shown. */
-static GtkWidget *deskno_configure(LXPanel *panel, GtkWidget *p, GtkWindow *parent)
+static GtkWidget *deskno_configure(LXPanel *panel, GtkWidget *p)
 {
     DesknoPlugin * dc = lxpanel_plugin_get_data(p);
     GtkWidget * dlg = lxpanel_generic_config_dlg(_("Desktop Number / Workspace Name"),
index efef864..b07a815 100644 (file)
@@ -348,7 +348,7 @@ static gboolean dirmenu_apply_configuration(gpointer user_data)
 }
 
 /* Callback when the configuration dialog is to be shown. */
-static GtkWidget *dirmenu_configure(LXPanel *panel, GtkWidget *p, GtkWindow *parent)
+static GtkWidget *dirmenu_configure(LXPanel *panel, GtkWidget *p)
 {
     DirMenuPlugin * dm = lxpanel_plugin_get_data(p);
     return lxpanel_generic_config_dlg(_("Directory Menu"),
index a4c1169..238fb6d 100644 (file)
@@ -858,7 +858,7 @@ static gboolean indicator_apply_configuration(gpointer user_data)
 }
 
 /* Callback when the configuration dialog is to be shown. */
-static GtkWidget *indicator_configure(LXPanel *panel, GtkWidget *p, GtkWindow *parent)
+static GtkWidget *indicator_configure(LXPanel *panel, GtkWidget *p)
 {
     IndicatorPlugin * indicator = lxpanel_plugin_get_data(p);
     GtkWidget * dlg = lxpanel_generic_config_dlg(_("Indicator applets"),
index 38afd37..eb9aa7e 100644 (file)
@@ -223,7 +223,7 @@ static gboolean kbled_apply_configuration(gpointer user_data)
 }
 
 /* Callback when the configuration dialog is to be shown. */
-static GtkWidget *kbled_configure(LXPanel *panel, GtkWidget *p, GtkWindow *parent)
+static GtkWidget *kbled_configure(LXPanel *panel, GtkWidget *p)
 {
     KeyboardLEDPlugin * kl = lxpanel_plugin_get_data(p);
     GtkWidget * dlg = lxpanel_generic_config_dlg(_("Keyboard LED"),
index 7ad6f86..da80e6c 100644 (file)
@@ -1319,7 +1319,7 @@ static void on_menu_view_row_activated(GtkTreeView *tree_view, GtkTreePath *path
 }
 
 /* Callback when the configuration dialog is to be shown. */
-static GtkWidget *launchtaskbar_configure(LXPanel *panel, GtkWidget *p, GtkWindow *parent)
+static GtkWidget *launchtaskbar_configure(LXPanel *panel, GtkWidget *p)
 {
     LaunchTaskBarPlugin *ltbp = lxpanel_plugin_get_data(p);
 
index f45aad2..f6769c6 100644 (file)
@@ -1013,7 +1013,7 @@ static gboolean apply_config(gpointer user_data)
     return FALSE;
 }
 
-static GtkWidget *menu_config(LXPanel *panel, GtkWidget *p, GtkWindow *parent)
+static GtkWidget *menu_config(LXPanel *panel, GtkWidget *p)
 {
     menup* menu = lxpanel_plugin_get_data(p);
     return lxpanel_generic_config_dlg(_("Menu"), panel, apply_config, p,
index d5bbf7a..1d5cd43 100644 (file)
@@ -688,7 +688,7 @@ monitors_destructor(gpointer user_data)
 
 
 static GtkWidget *
-monitors_config (LXPanel *panel, GtkWidget *p, GtkWindow *parent)
+monitors_config (LXPanel *panel, GtkWidget *p)
 {
     ENTER;
 
index 7d98f25..4ecb9fc 100644 (file)
@@ -146,7 +146,7 @@ static gboolean apply_config(gpointer user_data)
     return FALSE;
 }
 
-static GtkWidget *netstatus_config(LXPanel *panel, GtkWidget *p, GtkWindow *parent)
+static GtkWidget *netstatus_config(LXPanel *panel, GtkWidget *p)
 {
     GtkWidget* dlg;
     netstatus *ns = lxpanel_plugin_get_data(p);
index 043dd07..2dc7fc5 100644 (file)
@@ -81,7 +81,7 @@ static GtkWidget *pager_constructor(LXPanel *panel, config_setting_t *settings)
 }
 
 /* this is a modified version of patch from Lubuntu */
-static GtkWidget *pager_configure(LXPanel *panel, GtkWidget *instance, GtkWindow *parent)
+static GtkWidget *pager_configure(LXPanel *panel, GtkWidget *instance)
 {
     if (configure_command)
         fm_launch_command_simple(NULL, NULL, G_APP_INFO_CREATE_NONE,
index 7f6afc2..353288b 100644 (file)
@@ -79,7 +79,7 @@ static gboolean space_apply_configuration(gpointer user_data)
 }
 
 /* Callback when the configuration dialog is to be shown. */
-static GtkWidget *space_configure(LXPanel *panel, GtkWidget *instance, GtkWindow *parent)
+static GtkWidget *space_configure(LXPanel *panel, GtkWidget *instance)
 {
     SpacePlugin * sp = lxpanel_plugin_get_data(instance);
     GtkWidget * dlg;
index a252eaa..49181f4 100644 (file)
@@ -507,7 +507,7 @@ thermal_constructor(LXPanel *panel, config_setting_t *settings)
     RET(p);
 }
 
-static GtkWidget *config(LXPanel *panel, GtkWidget *p, GtkWindow *parent)
+static GtkWidget *config(LXPanel *panel, GtkWidget *p)
 {
     ENTER;
 
index 3b3e757..2f5e0c3 100644 (file)
@@ -574,7 +574,7 @@ static void volumealsa_destructor(gpointer user_data)
 
 /* Callback when the configuration dialog is to be shown. */
 
-static GtkWidget *volumealsa_configure(LXPanel *panel, GtkWidget *p, GtkWindow *parent)
+static GtkWidget *volumealsa_configure(LXPanel *panel, GtkWidget *p)
 {
     const gchar *command_line = NULL;
 
index 5005323..9e08522 100644 (file)
@@ -217,7 +217,7 @@ static gboolean wincmd_apply_configuration(gpointer user_data)
 }
 
 /* Callback when the configuration dialog is to be shown. */
-static GtkWidget *wincmd_configure(LXPanel *panel, GtkWidget *p, GtkWindow *parent)
+static GtkWidget *wincmd_configure(LXPanel *panel, GtkWidget *p)
 {
     WinCmdPlugin * wc = lxpanel_plugin_get_data(p);
     return lxpanel_generic_config_dlg(_("Minimize All Windows"),
index 12ccadf..e42f81a 100644 (file)
@@ -1192,7 +1192,7 @@ static void xkb_settings_fill_layout_tree_model_with_config(XkbPlugin *p_xkb)
 }
 
 /* Callback when the configuration dialog is to be shown. */
-static GtkWidget *xkb_configure(LXPanel *panel, GtkWidget *p, GtkWindow *parent)
+static GtkWidget *xkb_configure(LXPanel *panel, GtkWidget *p)
 {
     XkbPlugin * p_xkb = lxpanel_plugin_get_data(p);
     gchar       markup_str[MAX_MARKUP_LEN];
index 3a2b900..5621210 100644 (file)
@@ -199,10 +199,10 @@ void _unload_modules(void);
 GtkWidget *lxpanel_add_plugin(LXPanel *p, const char *name, config_setting_t *cfg, gint at);
 GHashTable *lxpanel_get_all_types(void); /* transfer none */
 
-GQuark lxpanel_plugin_qinit; /* access to LXPanelPluginInit data */
+extern GQuark lxpanel_plugin_qinit; /* access to LXPanelPluginInit data */
 #define PLUGIN_CLASS(_i) ((LXPanelPluginInit*)g_object_get_qdata(G_OBJECT(_i),lxpanel_plugin_qinit))
 
-GQuark lxpanel_plugin_qconf; /* access to congig_setting_t data */
+extern GQuark lxpanel_plugin_qconf; /* access to congig_setting_t data */
 
 #define PLUGIN_PANEL(_i) ((LXPanel*)gtk_widget_get_toplevel(_i))