Break the 'if' onto two lines and don't bother calling jabber_id_free()

Mon, 13 Jul 2009 18:44:28 +0000

author
Mark Doliner <markdoliner@pidgin.im>
date
Mon, 13 Jul 2009 18:44:28 +0000
changeset 27782
68314a2ac01b
parent 27781
f1b08fca9a9d
child 27783
902f03a8f3db

Break the 'if' onto two lines and don't bother calling jabber_id_free()
if jid is null

libpurple/protocols/jabber/message.c file | annotate | diff | comparison | revisions
--- a/libpurple/protocols/jabber/message.c	Mon Jul 13 17:01:52 2009 +0000
+++ b/libpurple/protocols/jabber/message.c	Mon Jul 13 18:44:28 2009 +0000
@@ -644,10 +644,10 @@
 
 							if (jid) {
 								chat = jabber_chat_find(js, jid->node, jid->domain);
-								if (chat) conv = chat->conv;
+								if (chat)
+									conv = chat->conv;
+								jabber_id_free(jid);
 							}
-
-							jabber_id_free(jid);
 						} else if (jm->type == JABBER_MESSAGE_NORMAL ||
 						           jm->type == JABBER_MESSAGE_CHAT) {
 							conv =

mercurial