Mercurial > pidgin.yaz
comparison src/protocols/irc/msgs.c @ 13666:915f26ce1d07
[gaim-migrate @ 16068]
Suppress CID 108.
Event assign_zero: Variable "(irc)->roomlist" assigned value 0.
278 irc->roomlist = NULL;
Event var_deref_model: Variable "(irc)->roomlist" tracked as NULL was passed to a function that dereferences it.
288 gaim_roomlist_room_add_field(irc->roomlist, room, args[1]);
nosnilmot noted:
'name can't be both "323" and "322" at the same time, so this will never happen'
In the 323 case, this saves us a strcmp() that we know is useless, and adding the return is consistent with the 321 block above.
committer: Tailor Script <tailor@pidgin.im>
author | Richard Laager <rlaager@wiktel.com> |
---|---|
date | Thu, 20 Apr 2006 04:56:44 +0000 |
parents | e8e5931e7e8d |
children | 3c11054a3b80 |
comparison
equal
deleted
inserted
replaced
13665:ee026e289bc4 | 13666:915f26ce1d07 |
---|---|
274 | 274 |
275 if (!strcmp(name, "323")) { | 275 if (!strcmp(name, "323")) { |
276 gaim_roomlist_set_in_progress(irc->roomlist, FALSE); | 276 gaim_roomlist_set_in_progress(irc->roomlist, FALSE); |
277 gaim_roomlist_unref(irc->roomlist); | 277 gaim_roomlist_unref(irc->roomlist); |
278 irc->roomlist = NULL; | 278 irc->roomlist = NULL; |
279 return; | |
279 } | 280 } |
280 | 281 |
281 if (!strcmp(name, "322")) { | 282 if (!strcmp(name, "322")) { |
282 GaimRoomlistRoom *room; | 283 GaimRoomlistRoom *room; |
283 | 284 |