Simplify lxpanel_plugin_show_config_dialog() parameters - no Panel* is required.
authorAndriy Grytsenko <andrej@rep.kiev.ua>
Fri, 1 Aug 2014 22:41:42 +0000 (01:41 +0300)
committerAndriy Grytsenko <andrej@rep.kiev.ua>
Fri, 1 Aug 2014 22:53:53 +0000 (01:53 +0300)
src/panel.c
src/plugin.c
src/plugin.h
src/plugins/launchtaskbar.c

index 7cee1f2..e198fd3 100644 (file)
@@ -579,7 +579,7 @@ static void panel_popupmenu_config_plugin( GtkMenuItem* item, GtkWidget* plugin
 {
     Panel *panel = PLUGIN_PANEL(plugin);
 
-    lxpanel_plugin_show_config_dialog(panel, plugin);
+    lxpanel_plugin_show_config_dialog(plugin);
 
     /* FIXME: this should be more elegant */
     panel->config_changed = TRUE;
index 3aec412..d5f310e 100644 (file)
@@ -416,13 +416,15 @@ gboolean lxpanel_launch_path(Panel *panel, FmPath *path)
     return fm_launch_path_simple(NULL, NULL, path, _open_dir_in_file_manager, NULL);
 }
 
-void lxpanel_plugin_show_config_dialog(Panel* panel, GtkWidget* plugin)
+void lxpanel_plugin_show_config_dialog(GtkWidget* plugin)
 {
     LXPanelPluginInit *init = PLUGIN_CLASS(plugin);
+    Panel *panel = PLUGIN_PANEL(plugin);
     GtkWidget *dlg = panel->plugin_pref_dialog;
 
     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->topgwin));
     if (dlg)
         _panel_show_config_dialog(panel, plugin, dlg);
index f1e77b1..7254e58 100644 (file)
@@ -126,10 +126,10 @@ extern void lxpanel_plugin_adjust_popup_position(GtkWidget * popup, GtkWidget *
                        /* Helper to move popup windows away from the panel */
 extern void lxpanel_plugin_popup_set_position_helper(Panel * p, GtkWidget * near, GtkWidget * popup, gint * px, gint * py);
                        /* Helper for position-calculation callback for popup menus */
-extern void plugin_widget_set_background(GtkWidget * plugin, Panel * p);
+extern void plugin_widget_set_background(GtkWidget * widget, Panel * p);
                        /* Recursively set the background of all widgets on a panel background configuration change */
 extern gboolean lxpanel_launch_path(Panel *panel, FmPath *path);
-extern void lxpanel_plugin_show_config_dialog(Panel* panel, GtkWidget* plugin);
+extern void lxpanel_plugin_show_config_dialog(GtkWidget* plugin);
                        /* Calls config() callback and shows configuration window */
 
 G_END_DECLS
index fcbb3d9..6f56f81 100644 (file)
@@ -343,7 +343,7 @@ static gboolean launchbutton_press_event(GtkWidget * widget, GdkEventButton * ev
     if (event->button == 1 && event->type == GDK_BUTTON_PRESS) /* left button */
     {
         if (b->fi == NULL)  /* The bootstrap button */
-            lxpanel_plugin_show_config_dialog(b->p->panel, b->p->plugin);
+            lxpanel_plugin_show_config_dialog(b->p->plugin);
         else
             lxpanel_launch_path(b->p->panel, fm_file_info_get_path(b->fi));
     }