Mercurial > pidgin
changeset 22140:c69f472ff683
Apparently gcc gives a warning here even though it's perfectly valid
and more efficient.
| author | Mark Doliner <mark@kingant.net> |
|---|---|
| date | Thu, 17 Jan 2008 07:53:03 +0000 |
| parents | 8d01e41786ce |
| children | 04bdf1ff5d2a 1f3f2d2c9a2b |
| files | libpurple/protocols/oscar/oscar.c |
| diffstat | 1 files changed, 1 insertions(+), 5 deletions(-) [+] |
line wrap: on
line diff
--- a/libpurple/protocols/oscar/oscar.c Thu Jan 17 07:39:25 2008 +0000 +++ b/libpurple/protocols/oscar/oscar.c Thu Jan 17 07:53:03 2008 +0000 @@ -1789,11 +1789,7 @@ purple_prpl_got_user_status_deactive(account, info->sn, OSCAR_STATUS_ID_MOBILE); } - /* - * It's ok for us to use == instead of strcmp() here because - * status_id is set to one of these #defines a few lines above here. - */ - if (status_id == OSCAR_STATUS_ID_AVAILABLE) + if (strcmp(status_id, OSCAR_STATUS_ID_AVAILABLE) == 0) { char *message = NULL; char *itmsurl = NULL;