Mercurial > pidgin
comparison gtk/gtkmain.c @ 14995:0d9f69b23352
[gaim-migrate @ 17774]
The data for "plugin_action" is set with a destructor function. So freeing it
twice would cause a crash. The reason it's not crashing is, the children of the
container are not the menuitems, but GtkAccelLabel (or some such thing). So all
we are doing is freeing null. Also, when the menu is destroyed, all the
menuitems are also destroyed anyway. So get rid of the unrequired loop.
Also, isdigit expects a character, not a string.
committer: Tailor Script <tailor@pidgin.im>
author | Sadrul Habib Chowdhury <imadil@gmail.com> |
---|---|
date | Sat, 18 Nov 2006 05:30:57 +0000 |
parents | 633d4c25e82c |
children | f79e0f4df793 |
comparison
equal
deleted
inserted
replaced
14994:5d71039b20eb | 14995:0d9f69b23352 |
---|---|
733 * before anything has happened | 733 * before anything has happened |
734 */ | 734 */ |
735 for (accounts = gaim_accounts_get_all(); accounts != NULL; accounts = accounts->next) { | 735 for (accounts = gaim_accounts_get_all(); accounts != NULL; accounts = accounts->next) { |
736 GaimAccount *account = accounts->data; | 736 GaimAccount *account = accounts->data; |
737 if (!strcmp(gaim_account_get_protocol_id(account), "prpl-oscar")) { | 737 if (!strcmp(gaim_account_get_protocol_id(account), "prpl-oscar")) { |
738 if (isdigit(gaim_account_get_username(account))) | 738 if (isdigit(*gaim_account_get_username(account))) |
739 gaim_account_set_protocol_id(account, "prpl-icq"); | 739 gaim_account_set_protocol_id(account, "prpl-icq"); |
740 else | 740 else |
741 gaim_account_set_protocol_id(account, "prpl-aim"); | 741 gaim_account_set_protocol_id(account, "prpl-aim"); |
742 } | 742 } |
743 } | 743 } |