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;