comparison src/plugin.c @ 12992:9d8025ec5e23

[gaim-migrate @ 15345] Etan is right about the dependent plugin unloading. committer: Tailor Script <[e-mail]>

12991:c1b2ba353e85

gboolean
gaim_plugin_unload(GaimPlugin *plugin)
{
#ifdef GAIM_PLUGINS
	GList *l, *l_next;

	g_return_val_if_fail(plugin != NULL, FALSE);

	loaded_plugins = g_list_remove(loaded_plugins, plugin);
	if ((plugin->info != NULL) && GAIM_IS_PROTOCOL_PLUGIN(plugin))

	gaim_notify_close_with_handle(plugin);

	plugin->loaded = FALSE;

	/* Unload all plugins that depend on this plugin. */
	/* XXX Does this want to be while (plugin->dependent_plugins != NULL)
	 * instead of this loop? I fixed it to get l->next before calling
	 * gaim_plugin_unload (which can change the list here). This worked
	 * much better for me but I'm not 100% sure it will always work. */
	for (l = plugin->dependent_plugins; l != NULL; l = l_next)
	{
		const char * dep_name = (const char *)l->data;
		GaimPlugin *dep_plugin;

		l_next = l->next;

		dep_plugin = gaim_plugins_find_with_id(dep_name);

		if (dep_plugin != NULL && gaim_plugin_is_loaded(dep_plugin))
		{

12992:9d8025ec5e23

gboolean
gaim_plugin_unload(GaimPlugin *plugin)
{
#ifdef GAIM_PLUGINS
	GList *l;

	g_return_val_if_fail(plugin != NULL, FALSE);

	loaded_plugins = g_list_remove(loaded_plugins, plugin);
	if ((plugin->info != NULL) && GAIM_IS_PROTOCOL_PLUGIN(plugin))

	gaim_notify_close_with_handle(plugin);

	plugin->loaded = FALSE;

	/* Unload all plugins that depend on this plugin. */
	while ((l = plugin->dependent_plugins) != NULL)
	{
		const char * dep_name = (const char *)l->data;
		GaimPlugin *dep_plugin;

		dep_plugin = gaim_plugins_find_with_id(dep_name);

		if (dep_plugin != NULL && gaim_plugin_is_loaded(dep_plugin))
		{