pidgin: 2ea51012: All these processing functions are calle...

qulogic at pidgin.im qulogic at pidgin.im
Fri May 21 18:02:58 EDT 2010


-----------------------------------------------------------------
Revision: 2ea51012685842a20704335af767bb915bcdb874
Ancestor: 73581f49564a5040aa639bed28fed8f6f7db1094
Author: qulogic at pidgin.im
Date: 2010-04-24T20:39:06
Branch: im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/2ea51012685842a20704335af767bb915bcdb874

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

ChangeLog: 

All these processing functions are called from places where dc != NULL,
so there's no need to waste time checking that.

-------------- next part --------------
============================================================
--- libpurple/protocols/msn/directconn.c	f46ef076ad5762766d2e834e6ebb9fa082014cf4
+++ libpurple/protocols/msn/directconn.c	0488d981db916f818fd7789c494d2a9b7913ffdc
@@ -626,8 +626,6 @@ msn_dc_send_foo(MsnDirectConn *dc)
 	if (purple_debug_is_verbose())
 		purple_debug_info("msn", "msn_dc_send_foo %p\n", dc);
 
-	g_return_if_fail(dc != NULL);
-
 	p = msn_dc_new_packet();
 
 	p->length = 8;
@@ -644,8 +642,6 @@ msn_dc_send_handshake(MsnDirectConn *dc)
 	const gchar *h;
 	guint32 len;
 
-	g_return_if_fail(dc != NULL);
-
 	p = msn_dc_new_packet();
 
 	p->length = 4 + DC_PACKET_HEADER_SIZE;
@@ -675,8 +671,6 @@ msn_dc_send_handshake_reply(MsnDirectCon
 	const gchar *h;
 	guint32 len;
 
-	g_return_if_fail(dc != NULL);
-
 	p = msn_dc_new_packet();
 
 	p->length = 4 + DC_PACKET_HEADER_SIZE;


More information about the Commits mailing list