Changeset 2f73692

Timestamp:

2015-10-08T07:43:45Z (9 years ago)

Author:

dequis <dx@…>

Branches:

master

Children:

58b63de6

Parents:

687ec88

git-author:

dequis <dx@…> (08-10-15 07:40:03)

git-committer:

dequis <dx@…> (08-10-15 07:43:45)

Message:

irc_send_who: move the iu->flags check after iu has a value...

Moving this was suggested during the review of the multi-prefix patch as
ordering the sets of status_prefix... and it seemed reasonable to me.
Thanks valgrind.

File:

• irc_send.c (modified) (2 diffs)

irc_send.c

@@ -295 +295 @@
                 char status_prefix[3] = {0};
 
-                /* rfc1459 doesn't mention this: G means gone, H means here */
-                status_prefix[0] = iu->flags & IRC_USER_AWAY ? 'G' : 'H';
-
                 if (is_channel) {
                         irc_channel_user_t *icu = l->data;
@@ -305 +302 @@
                         iu = l->data;
                 }
+
+                /* rfc1459 doesn't mention this: G means gone, H means here */
+                status_prefix[0] = iu->flags & IRC_USER_AWAY ? 'G' : 'H';
 
                 irc_send_num(irc, 352, "%s %s %s %s %s %s :0 %s",