Mercurial > pidgin
comparison 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 |
comparison
equal
deleted
inserted
replaced
13105:e347b2217b1b | 13106:a0a4b44239e8 |
---|---|
1543 return plugins; | 1543 return plugins; |
1544 } | 1544 } |
1545 | 1545 |
1546 | 1546 |
1547 GaimPluginAction * | 1547 GaimPluginAction * |
1548 gaim_plugin_action_new(char* label, void (*callback)(GaimPluginAction *)) | 1548 gaim_plugin_action_new(const char* label, void (*callback)(GaimPluginAction *)) |
1549 { | 1549 { |
1550 GaimPluginAction *act = g_new0(GaimPluginAction, 1); | 1550 GaimPluginAction *act = g_new0(GaimPluginAction, 1); |
1551 | 1551 |
1552 act->label = label; | 1552 act->label = g_strdup(label); |
1553 act->callback = callback; | 1553 act->callback = callback; |
1554 | 1554 |
1555 return act; | 1555 return act; |
1556 } | 1556 } |
1557 | |
1558 void | |
1559 gaim_plugin_action_free(GaimPluginAction *action) | |
1560 { | |
1561 g_return_if_fail(action != NULL); | |
1562 | |
1563 g_free(action->label); | |
1564 g_free(action); | |
1565 } |