Mercurial > pidgin
diff src/plugin.c @ 13106:a0a4b44239e8
[gaim-migrate @ 15468]
I was reading the gettext man page and it pointed out that it should be typed as const char *, but it's char * to avoid warnings in code predating ANSI C. So, for the heck of it, I changed added a cast in internal.h. As it turns out, there was a lot of code that relied on this. In the interest of type safety, I've fixed all the warnings. I feel this improved a number of function signatures (in terms of typing clarity). Flame me if you object.
committer: Tailor Script <tailor@pidgin.im>
author | Richard Laager <rlaager@wiktel.com> |
---|---|
date | Thu, 02 Feb 2006 21:34:43 +0000 |
parents | 9d8025ec5e23 |
children | 7fd39c81d5e9 |
line wrap: on
line diff
--- a/src/plugin.c Thu Feb 02 20:03:17 2006 +0000 +++ b/src/plugin.c Thu Feb 02 21:34:43 2006 +0000 @@ -1545,12 +1545,21 @@ GaimPluginAction * -gaim_plugin_action_new(char* label, void (*callback)(GaimPluginAction *)) +gaim_plugin_action_new(const char* label, void (*callback)(GaimPluginAction *)) { GaimPluginAction *act = g_new0(GaimPluginAction, 1); - act->label = label; + act->label = g_strdup(label); act->callback = callback; return act; } + +void +gaim_plugin_action_free(GaimPluginAction *action) +{ + g_return_if_fail(action != NULL); + + g_free(action->label); + g_free(action); +}