pidgin: b4a46d69: I'm guessing this is what was intended h...

markdoliner at pidgin.im markdoliner at pidgin.im
Thu Dec 18 05:10:29 EST 2008


-----------------------------------------------------------------
Revision: b4a46d69ddab9c22a5dee690f0879c261f567fff
Ancestor: b035dc240e15fbfaeb51979cebc4872220637d46
Author: markdoliner at pidgin.im
Date: 2008-12-18T10:05:58
Branch: im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/b4a46d69ddab9c22a5dee690f0879c261f567fff

Modified files:
        libpurple/protocols/msn/soap.c

ChangeLog: 

I'm guessing this is what was intended here?
Fixes this warning:

soap.c: In function 'msn_soap_read_cb':
soap.c:524: warning: suggest parentheses around && within ||


-------------- next part --------------
============================================================
--- libpurple/protocols/msn/soap.c	471109ac485058a9b0dce16f874080883b214c89
+++ libpurple/protocols/msn/soap.c	40430f70e32880ef814c3f116e692f4e638b7fb6
@@ -521,7 +521,7 @@ msn_soap_read_cb(gpointer data, gint fd,
 	/* msn_soap_process could alter errno */
 	msn_soap_process(conn);
 
-	if (cnt < 0 && perrno != EAGAIN || cnt == 0) {
+	if ((cnt < 0 && perrno != EAGAIN) || cnt == 0) {
 		/* It's possible msn_soap_process closed the ssl connection */
 		if (conn->ssl) {
 			purple_ssl_close(conn->ssl);


More information about the Commits mailing list