Revert "WIP - Unload plugin commands"

This reverts commit a01eb5d08e1b39d60a6f8fc26e5a87ceb92ec18f.

21 files changed, 21 insertions, 162 deletions

diff --git a/src/command/cmd_ac.c b/src/command/cmd_ac.c
index 46c1d940..1dfc5cc0 100644
--- a/src/command/cmd_ac.c
+++ b/src/command/cmd_ac.c
@@ -702,7 +702,6 @@ cmd_ac_init(void)
 
     plugins_ac = autocomplete_new();
     autocomplete_add(plugins_ac, "load");
-    autocomplete_add(plugins_ac, "unload");
 
     sendfile_ac = autocomplete_new();
 
@@ -785,14 +784,6 @@ cmd_ac_remove(const char *const value)
     }
 }
 
-void
-cmd_ac_remove_help(const char *const value)
-{
-    if (help_ac) {
-        autocomplete_remove(help_ac, value);
-    }
-}
-
 gboolean
 cmd_ac_exists(char *cmd)
 {
diff --git a/src/command/cmd_ac.h b/src/command/cmd_ac.h
index 637ebcf6..b294fcd5 100644
--- a/src/command/cmd_ac.h
+++ b/src/command/cmd_ac.h
@@ -48,7 +48,6 @@ void cmd_ac_add_alias(ProfAlias *alias);
 void cmd_ac_add_alias_value(char *value);
 
 void cmd_ac_remove(const char *const value);
-void cmd_ac_remove_help(const char *const value);
 void cmd_ac_remove_alias_value(char *value);
 
 gboolean cmd_ac_exists(char *cmd);
diff --git a/src/command/cmd_defs.c b/src/command/cmd_defs.c
index 663a8948..dacf1c91 100644
--- a/src/command/cmd_defs.c
+++ b/src/command/cmd_defs.c
@@ -1979,16 +1979,13 @@ static struct cmd_t command_defs[] =
         CMD_NOTAGS
         CMD_SYN(
             "/plugins",
-            "/plugins load <plugin>",
-            "/plugins unload <plugin>")
+            "/plugins load <plugin>")
         CMD_DESC(
             "Manage plugins. Passing no arguments lists currently loaded plugins.")
         CMD_ARGS(
-            { "load <plugin>",       "Load a plugin." },
-            { "unload <plugin>",     "Unload a plugin." })
+            { "load <plugin>",       "Load a plugin." })
         CMD_EXAMPLES(
-            "/plugin load browser.py",
-            "/plugin unload pid.so")
+            "/plugin load browser.py")
     },
 
     { "/prefs",
diff --git a/src/command/cmd_funcs.c b/src/command/cmd_funcs.c
index 0d673248..7f3419e3 100644
--- a/src/command/cmd_funcs.c
+++ b/src/command/cmd_funcs.c
@@ -95,49 +95,6 @@ static gboolean _cmd_execute(ProfWin *window, const char *const command, const c
 static gboolean _cmd_execute_default(ProfWin *window, const char *inp);
 static gboolean _cmd_execute_alias(ProfWin *window, const char *const inp, gboolean *ran);
 
-void
-command_help_free(CommandHelp *help)
-{
-    free(help->desc);
-
-    if (help->tags) {
-        int i = 0;
-        while (i < 20 && help->tags[i]) {
-            free(help->tags[i]);
-            i++;
-        }
-        free(help->tags);
-    }
-
-    if (help->synopsis) {
-        int i = 0;
-        while (i < 50 && help->synopsis[i]) {
-            free(help->synopsis[i]);
-            i++;
-        }
-        free(help->synopsis);
-    }
-
-    if (help->examples) {
-        int i = 0;
-        while (i < 20 && help->examples[i]) {
-            free(help->examples[i]);
-            i++;
-        }
-        free(help->examples);
-    }
-
-    if (help->args) {
-        int i = 0;
-        while (i < 120 && help->args[i]) {
-            free(help->args[i][0]);
-            free(help->args[i][1]);
-            free(help->args[i]);
-        }
-        free(help->args);
-    }
-}
-
 /*
  * Take a line of input and process it, return TRUE if profanity is to
  * continue, FALSE otherwise
@@ -3852,7 +3809,7 @@ cmd_form(ProfWin *window, const char *const command, gchar **args)
         } else {
             mucconfwin_form_help(confwin);
 
-            gchar **help_text = NULL;
+            const gchar **help_text = NULL;
             Command *command = cmd_get("/form");
 
             if (command) {
@@ -6082,16 +6039,6 @@ cmd_plugins(ProfWin *window, const char *const command, gchar **args)
         }
 
         return TRUE;
-    } else if (g_strcmp0(args[0], "unload") == 0) {
-        if (args[1] == NULL) {
-            cons_bad_cmd_usage(command);
-            return TRUE;
-        }
-        plugins_unload(args[1]);
-        prefs_remove_plugin(args[1]);
-        cons_show("Unloaded plugin: %s", args[1]);
-
-        return TRUE;
     } else {
         GSList *plugins = plugins_get_list();
         GSList *curr = plugins;
diff --git a/src/command/cmd_funcs.h b/src/command/cmd_funcs.h
index c68efe43..52b9946e 100644
--- a/src/command/cmd_funcs.h
+++ b/src/command/cmd_funcs.h
@@ -39,11 +39,11 @@
 
 // Command help strings
 typedef struct cmd_help_t {
-    gchar *tags[20];
-    gchar *synopsis[50];
-    gchar *desc;
-    gchar *args[128][2];
-    gchar *examples[20];
+    const gchar *tags[20];
+    const gchar *synopsis[50];
+    const gchar *desc;
+    const gchar *args[128][2];
+    const gchar *examples[20];
 } CommandHelp;
 
 /*
@@ -69,7 +69,6 @@ typedef struct cmd_t {
     CommandHelp help;
 } Command;
 
-void command_help_free(CommandHelp *help);
 
 gboolean cmd_process_input(ProfWin *window, char *inp);
 void cmd_execute_connect(ProfWin *window, const char *const account);
diff --git a/src/config/preferences.c b/src/config/preferences.c
index 636bd33d..9381d014 100644
--- a/src/config/preferences.c
+++ b/src/config/preferences.c
@@ -632,13 +632,6 @@ prefs_add_plugin(const char *const name)
 }
 
 void
-prefs_remove_plugin(const char *const name)
-{
-    conf_string_list_remove(prefs, "plugins", "load", name);
-    _save_prefs();
-}
-
-void
 prefs_free_plugins(gchar **plugins)
 {
     g_strfreev(plugins);
diff --git a/src/config/preferences.h b/src/config/preferences.h
index 3111f81e..a7b84dfe 100644
--- a/src/config/preferences.h
+++ b/src/config/preferences.h
@@ -191,7 +191,6 @@ void prefs_set_autoxa_time(gint value);
 gchar** prefs_get_plugins(void);
 void prefs_free_plugins(gchar **plugins);
 void prefs_add_plugin(const char *const name);
-void prefs_remove_plugin(const char *const name);
 
 char prefs_get_otr_char(void);
 void prefs_set_otr_char(char ch);
diff --git a/src/plugins/api.c b/src/plugins/api.c
index 90272684..326ec61d 100644
--- a/src/plugins/api.c
+++ b/src/plugins/api.c
@@ -106,12 +106,11 @@ api_cons_bad_cmd_usage(const char *const cmd)
 }
 
 void
-api_register_command(char *plugin_name, char *command_name, int min_args, int max_args,
+api_register_command(const char *command_name, int min_args, int max_args,
     const char **synopsis, const char *description, const char *arguments[][2], const char **examples, void *callback,
     void(*callback_func)(PluginCommand *command, gchar **args))
 {
     PluginCommand *command = malloc(sizeof(PluginCommand));
-    command->plugin_name = plugin_name;
     command->command_name = command_name;
     command->min_args = min_args;
     command->max_args = max_args;
diff --git a/src/plugins/api.h b/src/plugins/api.h
index 6bbcfa5c..a835cd8b 100644
--- a/src/plugins/api.h
+++ b/src/plugins/api.h
@@ -50,7 +50,7 @@ gboolean api_current_win_is_console(void);
 char* api_get_current_nick(void);
 char** api_get_current_occupants(void);
 
-void api_register_command(char *plugin_name, char *command_name, int min_args, int max_args,
+void api_register_command(const char *command_name, int min_args, int max_args,
     const char **synopsis, const char *description, const char *arguments[][2], const char **examples,
     void *callback, void(*callback_func)(PluginCommand *command, gchar **args));
 void api_register_timed(void *callback, int interval_seconds,
diff --git a/src/plugins/c_api.c b/src/plugins/c_api.c
index 4e5900b9..b5624946 100644
--- a/src/plugins/c_api.c
+++ b/src/plugins/c_api.c
@@ -80,7 +80,7 @@ c_api_cons_bad_cmd_usage(const char *const cmd)
 }
 
 static void
-c_api_register_command(const char *filename, char *command_name, int min_args, int max_args,
+c_api_register_command(const char *filename, const char *command_name, int min_args, int max_args,
     const char **synopsis, const char *description, const char *arguments[][2], const char **examples,
     void(*callback)(char **args))
 {
@@ -89,7 +89,7 @@ c_api_register_command(const char *filename, char *command_name, int min_args, i
 
     CommandWrapper *wrapper = malloc(sizeof(CommandWrapper));
     wrapper->func = callback;
-    api_register_command(plugin_name, command_name, min_args, max_args, synopsis,
+    api_register_command(command_name, min_args, max_args, synopsis,
         description, arguments, examples, wrapper, c_command_callback);
 }
 
diff --git a/src/plugins/callbacks.c b/src/plugins/callbacks.c
index 8960738f..2572f7a4 100644
--- a/src/plugins/callbacks.c
+++ b/src/plugins/callbacks.c
@@ -78,40 +78,6 @@ callbacks_add_command(PluginCommand *command)
 }
 
 void
-_command_destroy(PluginCommand *command)
-{
-    free(command->command_name);
-    free(command->plugin_name);
-    command_help_free(command->help);
-    // TODO free callback
-}
-
-void
-callbacks_remove_commands(const char *const plugin_name)
-{
-    GSList *items_to_remove = NULL;
-    GSList *curr = p_commands;
-    while (curr) {
-        PluginCommand *command = curr->data;
-        if (g_strcmp0(command->plugin_name, plugin_name) == 0) {
-            cmd_ac_remove(command->command_name);
-            cmd_ac_remove_help(&command->command_name[1]);
-            items_to_remove = g_slist_append(items_to_remove, curr);
-        }
-        curr = g_slist_next(curr);
-    }
-
-    curr = items_to_remove;
-    while (curr) {
-        GSList *item = curr->data;
-        PluginCommand *command = item->data;
-        _command_destroy(command);
-        p_commands = g_slist_remove_link(p_commands, item);
-        curr = g_slist_next(curr);
-    }
-}
-
-void
 callbacks_add_timed(PluginTimedFunction *timed_function)
 {
     p_timed_functions = g_slist_append(p_timed_functions, timed_function);
diff --git a/src/plugins/callbacks.h b/src/plugins/callbacks.h
index a8253e21..9b175d0a 100644
--- a/src/plugins/callbacks.h
+++ b/src/plugins/callbacks.h
@@ -40,8 +40,7 @@
 #include "command/cmd_defs.h"
 
 typedef struct p_command {
-    char *plugin_name;
-    char *command_name;
+    const char *command_name;
     int min_args;
     int max_args;
     CommandHelp *help;
@@ -66,7 +65,6 @@ void callbacks_init(void);
 void callbacks_close(void);
 
 void callbacks_add_command(PluginCommand *command);
-void callbacks_remove_commands(const char *const plugin_name);
 void callbacks_add_timed(PluginTimedFunction *timed_function);
 void callbacks_add_window_handler(const char *tag, PluginWindowCallback *window_callback);
 void * callbacks_get_window_handler(const char *tag);
diff --git a/src/plugins/plugins.c b/src/plugins/plugins.c
index 3395c77f..5757a174 100644
--- a/src/plugins/plugins.c
+++ b/src/plugins/plugins.c
@@ -172,33 +172,6 @@ plugins_load(const char *const name)
     }
 }
 
-gboolean
-plugins_unload(const char *const name)
-{
-    GSList *found = g_slist_find_custom(plugins, name, (GCompareFunc)_find_by_name);
-    if (!found) {
-        log_info("Failed to unload plugin: %s, plugin not currently loaded", name);
-        return FALSE;
-    }
-
-    plugins = g_slist_remove_link(plugins, found);
-
-#ifdef HAVE_PYTHON
-    if (g_str_has_suffix(name, ".py")) {
-        python_plugin_destroy(found->data);
-    }
-#endif
-#ifdef HAVE_C
-    if (g_str_has_suffix(name, ".so")) {
-        c_plugin_destroy(found->data);
-    }
-#endif
-
-    g_slist_free(found);
-
-    return TRUE;
-}
-
 GSList *
 plugins_get_list(void)
 {
diff --git a/src/plugins/plugins.h b/src/plugins/plugins.h
index 19ac269d..b03248ab 100644
--- a/src/plugins/plugins.h
+++ b/src/plugins/plugins.h
@@ -105,7 +105,6 @@ void plugins_reset_autocomplete(void);
 void plugins_shutdown(void);
 
 gboolean plugins_load(const char *const name);
-gboolean plugins_unload(const char *const name);
 
 void plugins_on_start(void);
 void plugins_on_shutdown(void);
diff --git a/src/plugins/profapi.c b/src/plugins/profapi.c
index 617fd9f0..d80e2690 100644
--- a/src/plugins/profapi.c
+++ b/src/plugins/profapi.c
@@ -42,7 +42,7 @@ int (*prof_cons_show)(const char * const message) = NULL;
 int (*prof_cons_show_themed)(const char *const group, const char *const item, const char *const def, const char *const message) = NULL;
 int (*prof_cons_bad_cmd_usage)(const char *const cmd) = NULL;
 
-void (*_prof_register_command)(const char *filename, char *command_name, int min_args, int max_args,
+void (*_prof_register_command)(const char *filename, const char *command_name, int min_args, int max_args,
     const char **synopsis, const char *description, const char *arguments[][2], const char **examples,
     void(*callback)(char **args)) = NULL;
 
diff --git a/src/plugins/profapi.h b/src/plugins/profapi.h
index 7f26c33f..8c41b2f2 100644
--- a/src/plugins/profapi.h
+++ b/src/plugins/profapi.h
@@ -46,7 +46,7 @@ int (*prof_cons_show)(const char * const message);
 int (*prof_cons_show_themed)(const char *const group, const char *const item, const char *const def, const char *const message);
 int (*prof_cons_bad_cmd_usage)(const char *const cmd);
 
-void (*_prof_register_command)(const char *filename, char *command_name, int min_args, int max_args,
+void (*_prof_register_command)(const char *filename, const char *command_name, int min_args, int max_args,
     const char **synopsis, const char *description, const char *arguments[][2], const char **examples,
     void(*callback)(char **args));
 
diff --git a/src/plugins/python_api.c b/src/plugins/python_api.c
index 356a2854..87a2b4d2 100644
--- a/src/plugins/python_api.c
+++ b/src/plugins/python_api.c
@@ -105,7 +105,7 @@ python_api_cons_bad_cmd_usage(PyObject *self, PyObject *args)
 static PyObject*
 python_api_register_command(PyObject *self, PyObject *args)
 {
-    char *command_name = NULL;
+    const char *command_name = NULL;
     int min_args = 0;
     int max_args = 0;
     PyObject *synopsis = NULL;
@@ -165,7 +165,7 @@ python_api_register_command(PyObject *self, PyObject *args)
         c_examples[len] = NULL;
 
         allow_python_threads();
-        api_register_command(plugin_name, command_name, min_args, max_args, c_synopsis,
+        api_register_command(command_name, min_args, max_args, c_synopsis,
             description, c_arguments, c_examples, p_callback, python_command_callback);
         disable_python_threads();
     }
diff --git a/src/plugins/python_plugins.c b/src/plugins/python_plugins.c
index 8e8f72a7..7861f484 100644
--- a/src/plugins/python_plugins.c
+++ b/src/plugins/python_plugins.c
@@ -888,7 +888,6 @@ python_check_error(void)
 void
 python_plugin_destroy(ProfPlugin *plugin)
 {
-    callbacks_remove_commands(plugin->name);
     disable_python_threads();
     free(plugin->name);
     Py_XDECREF(plugin->module);
diff --git a/src/ui/core.c b/src/ui/core.c
index c687cba6..3a89008f 100644
--- a/src/ui/core.c
+++ b/src/ui/core.c
@@ -1230,7 +1230,7 @@ ui_handle_room_config_submit_result_error(const char *const roomjid, const char
 }
 
 void
-ui_show_lines(ProfWin *window, gchar** lines)
+ui_show_lines(ProfWin *window, const gchar** lines)
 {
     if (lines) {
         int i;
diff --git a/src/ui/ui.h b/src/ui/ui.h
index 5cf8cb31..c15af910 100644
--- a/src/ui/ui.h
+++ b/src/ui/ui.h
@@ -110,7 +110,7 @@ void ui_goodbye_title(void);
 void ui_handle_room_configuration_form_error(const char *const roomjid, const char *const message);
 void ui_handle_room_config_submit_result(const char *const roomjid);
 void ui_handle_room_config_submit_result_error(const char *const roomjid, const char *const message);
-void ui_show_lines(ProfWin *window, gchar** lines);
+void ui_show_lines(ProfWin *window, const gchar** lines);
 void ui_redraw_all_room_rosters(void);
 void ui_show_all_room_rosters(void);
 void ui_hide_all_room_rosters(void);
diff --git a/tests/unittests/ui/stub_ui.c b/tests/unittests/ui/stub_ui.c
index b0635fb8..5e86799a 100644
--- a/tests/unittests/ui/stub_ui.c
+++ b/tests/unittests/ui/stub_ui.c
@@ -265,7 +265,7 @@ void mucconfwin_show_form(ProfMucConfWin *confwin) {}
 void mucconfwin_show_form_field(ProfMucConfWin *confwin, DataForm *form, char *tag) {}
 void mucconfwin_form_help(ProfMucConfWin *confwin) {}
 void mucconfwin_field_help(ProfMucConfWin *confwin, char *tag) {}
-void ui_show_lines(ProfWin *window, gchar** lines) {}
+void ui_show_lines(ProfWin *window, const gchar** lines) {}
 void ui_redraw_all_room_rosters(void) {}
 void ui_show_all_room_rosters(void) {}
 void ui_hide_all_room_rosters(void) {}