* Implement likely()/unlikely() blocks from ratbox/linux

This will probalby be mostly pointless, but should help in some
cases with caching/pipelining in critical components and common
cases.

src/common.h

@@ -77,5 +77,36 @@
 #define BIT30   (uint32_t) 0x040000000
 #define BIT31   (uint32_t) 0x080000000
 
+/* Thanks libratbox */
+#ifdef __GNUC__
+
+#ifdef likely
+#undef likely
+#endif
+#ifdef unlikely
+#undef unlikely
+#endif
+
+#if __GNUC__ == 2 && __GNUC_MINOR__ < 96
+# define __builtin_expect(x, expected_value) (x)
+#endif
+
+#define likely(x)       __builtin_expect(!!(x), 1)
+#define unlikely(x)     __builtin_expect(!!(x), 0)
+
+#else /* !__GNUC__ */
+
+#define UNUSED(x) x
+
+#ifdef likely
+#undef likely
+#endif
+#ifdef unlikely
+#undef unlikely
+#endif
+#define likely(x)    (x)
+#define unlikely(x)  (x)
+#endif
+
 
 #endif				/* _COMMON_H */

src/crypt.c

@@ -25,7 +25,7 @@ char *encrypt_string(const char *keydata, char *in)
 
   len = strlen(in);
   bdata = aes_encrypt_ecb_binary(keydata, (unsigned char *) in, &len);
-  if (keydata && *keydata) {
+  if (likely(keydata && *keydata)) {
     res = b64enc(bdata, len);
     OPENSSL_cleanse(bdata, len);
     free(bdata);
@@ -88,7 +88,7 @@ char *decrypt_string(const char *keydata, char *in)
   size_t len = strlen(in);
   char *buf = NULL, *res = NULL;
 
-  if (keydata && *keydata) {
+  if (likely(keydata && *keydata)) {
     buf = b64dec((const unsigned char *) in, &len);
     res = (char *) aes_decrypt_ecb_binary(keydata, (unsigned char *) buf, &len);
     OPENSSL_cleanse(buf, len);

src/dcc.c

@@ -370,7 +370,7 @@ cont_link(int idx, char *buf, int ii)
 {
   /* If we're already connected somewhere, unlink and idle a sec */
   for (int i = 0; i < dcc_total; i++) {
-    if (dcc[i].type && (dcc[i].type == &DCC_BOT) && (!bot_aggressive_to(dcc[i].user))) {
+    if (unlikely(dcc[i].type && (dcc[i].type == &DCC_BOT) && (!bot_aggressive_to(dcc[i].user)))) {
       putlog(LOG_BOTS, "*", "Unlinking %s - restructure", dcc[i].nick);
       botnet_send_unlinked(i, dcc[i].nick, "Restructure");
       killsock(dcc[i].sock);
@@ -1222,16 +1222,16 @@ dcc_chat(int idx, char *buf, int len)
     else
       *d = 0;
 
-    if (u_pass_match(dcc[idx].user, buf)) {     /* user said their password :) */
+    if (unlikely(u_pass_match(dcc[idx].user, buf))) {     /* user said their password :) */
       dprintf(idx, "Sure you want that going to the partyline? ;) (msg to partyline halted.)\n");
-    } else if (!strncmp(buf, STR("+Auth "), 6)) {    /* ignore extra +Auth lines */
+    } else if (unlikely(!strncmp(buf, STR("+Auth "), 6))) {    /* ignore extra +Auth lines */
     } else if ((!strncmp(buf, settings.dcc_prefix, strlen(settings.dcc_prefix))) || (dcc[idx].u.chat->channel < 0)) {
       if (!strncmp(buf, settings.dcc_prefix, strlen(settings.dcc_prefix)) && (dcc[idx].u.chat->channel >= 0))        /* strip '.' out */
         buf++;
       v = newsplit(&buf);
       rmspace(buf);
       check_bind_dcc(v, idx, buf);
-    } else if (buf[0] == ',') {
+    } else if (unlikely(buf[0] == ',')) {
       int me = 0;
 
       if ((buf[1] == 'm') && (buf[2] == 'e') && buf[3] == ' ')
@@ -1257,7 +1257,7 @@ dcc_chat(int idx, char *buf, int len)
         }
        }
       }
-    } else if (buf[0] == '\'') {
+    } else if (unlikely(buf[0] == '\'')) {
       int me = 0;
 
       if ((buf[1] == 'm') && (buf[2] == 'e') && ((buf[3] == ' ') || (buf[3] == '\'') || (buf[3] == ',')))
@@ -1377,7 +1377,7 @@ dcc_telnet(int idx, char *buf, int ii)
   int i;
   char x[1024] = "";
 
-  if (dcc_total + 1 > max_dcc) {
+  if (unlikely(dcc_total + 1 > max_dcc)) {
     int j;
 
     j = answer(dcc[idx].sock, s, &ip, &port, 0);
@@ -1392,7 +1392,7 @@ dcc_telnet(int idx, char *buf, int ii)
 
   while ((sock == -1) && (errno == EAGAIN))
     sock = answer(dcc[idx].sock, s, &ip, &port, 0);
-  if (sock < 0) {
+  if (unlikely(sock < 0)) {
     putlog(LOG_MISC, "*", "Failed TELNET incoming (%s)", strerror(errno));
 //    killsock(dcc[idx].sock);
     return;
@@ -1433,7 +1433,7 @@ dcc_telnet(int idx, char *buf, int ii)
   // Are they ignored by IP?
   simple_snprintf(x, sizeof(x), "-telnet!telnet@%s", iptostr(htonl(ip)));
 
-  if (match_ignore(x) || detect_telnet_flood(x)) {
+  if (unlikely(match_ignore(x) || detect_telnet_flood(x))) {
     putlog(LOG_DEBUG, "*", "Ignored telnet connection from: %s", x);
     killsock(sock);
     return;

src/dccutil.c

@@ -211,7 +211,7 @@ colorbuf(char *buf, size_t len, int idx, size_t bufsiz)
  */
 void dumplots(int idx, const char *prefix, const char *data)
 {
-  if (!*data) {
+  if (unlikely(!*data)) {
     dprintf(idx, "%s\n", prefix);
     return;
   }
@@ -268,7 +268,7 @@ dprintf(int idx, const char *format, ...)
   int vlen = egg_vsnprintf(buf, sizeof(buf), format, va);
   va_end(va);
 
-  if (vlen < 0) {
+  if (unlikely(vlen < 0)) {
     // Error parsing format..
     return;
   }
@@ -282,10 +282,10 @@ dprintf(int idx, const char *format, ...)
 
 /* this is for color on dcc :P */
 
-  if (idx < 0) {
+  if (unlikely(idx < 0)) {
     tputs(-idx, buf, len);
   } else if (idx > 0x7FF0) {
-    if (idx == DP_STDOUT || idx == DP_STDOUT) {
+    if (unlikely(idx == DP_STDOUT || idx == DP_STDOUT)) {
       len = colorbuf(buf, len, -1, sizeof(buf));
     }
 
@@ -311,7 +311,7 @@ dprintf(int idx, const char *format, ...)
           break;
         len -= remove_crlf_r(buf);
 
-        if ((idx == DP_DUMP || (floodless && idx != DP_CACHE))) {
+        if (unlikely((idx == DP_DUMP || (floodless && idx != DP_CACHE)))) {
          if (serv != -1) {
            if (debug_output)
              putlog(LOG_SRVOUT, "@", "[d->] %s", buf);
@@ -330,9 +330,9 @@ dprintf(int idx, const char *format, ...)
       buf[len - 1] = '\n';
       buf[len] = 0;
     }
-    if (dcc[idx].simul >= 0 && !dcc[idx].irc) {
+    if (unlikely(dcc[idx].simul >= 0 && !dcc[idx].irc)) {
       bounce_simul(idx, buf);
-    } else if (dcc[idx].irc) {
+    } else if (unlikely(dcc[idx].irc)) {
 //      size_t size = strlen(dcc[idx].simulbot) + strlen(buf) + 20;
 //      char *ircbuf = (char *) my_calloc(1, size);
 
@@ -709,7 +709,7 @@ new_dcc(struct dcc_table *type, int xtra_size)
       break;
 
   /* we managed to get to the end of the list! */
-  if (i == dcc_total) {
+  if (unlikely(i == dcc_total)) {
     i = dcc_total;
     dcc_total++;
   }

src/enclink.c

@@ -35,7 +35,7 @@ static void ghost_link_case(int idx, direction_t direction)
 {
   int snum = findanysnum(dcc[idx].sock);
 
-  if (snum >= 0) {
+  if (likely(snum >= 0)) {
     char initkey[33] = "", *tmp2 = NULL;
     char *keyp = NULL, *nick1 = NULL, *nick2 = NULL;
     port_t port = 0;

src/main.c

@@ -915,9 +915,9 @@ int main(int argc, char **argv)
     xx = sockgets(buf, &i);
 
     if (xx >= 0) {		/* Non-error */
-      for (idx = 0; idx < dcc_total; idx++) {
+      for (idx = 0; idx < dcc_total; idx++) {//FIXME: This is looking up the idx for the sock, looping quite a bit, needs to be O(1)
 	if (dcc[idx].type && dcc[idx].sock == xx) {
-	  if (dcc[idx].type && dcc[idx].type->activity) {
+	  if (likely(dcc[idx].type && dcc[idx].type->activity)) {
 	    /* Traffic stats */
 	    if (dcc[idx].type->name) {
 	      if (!strncmp(dcc[idx].type->name, "BOT", 3))
@@ -943,13 +943,13 @@ int main(int argc, char **argv)
 	  break;
 	}
       }
-    } else if (xx == -1) {	/* EOF from someone */
-      if (i == STDOUT && !backgrd)
+    } else if (unlikely(xx == -1)) {	/* EOF from someone */
+      if (unlikely(i == STDOUT && !backgrd))
 	fatal(STR("END OF FILE ON TERMINAL"), 0);
-      for (idx = 0; idx < dcc_total; idx++) {
+      for (idx = 0; idx < dcc_total; idx++) {//FIXME: Same here
 	if (dcc[idx].type && dcc[idx].sock == i) {
           sdprintf(STR("EOF on '%s' idx: %d"), dcc[idx].type ? dcc[idx].type->name : "unknown", idx);
-	  if (dcc[idx].type->eof)
+	  if (likely(dcc[idx].type->eof))
 	    dcc[idx].type->eof(idx);
 	  else {
 	    putlog(LOG_MISC, "*",
@@ -961,12 +961,12 @@ int main(int argc, char **argv)
 	  idx = dcc_total + 1;
 	}
       }
-      if (idx == dcc_total) {
+      if (unlikely(idx == dcc_total)) {
 	putlog(LOG_MISC, "*", STR("(@) EOF socket %d, not a dcc socket, not anything."), i);
 	close(i);
 	killsock(i);
       }
-    } else if (xx == -2 && errno != EINTR) {	/* select() error */
+    } else if (unlikely(xx == -2 && errno != EINTR)) {	/* select() error */
       putlog(LOG_MISC, "*", STR("* Socket error #%d; recovering."), errno);
       for (i = 0; i < dcc_total; i++) {
 	if (dcc[i].type && dcc[i].sock != -1 && (fcntl(dcc[i].sock, F_GETFD, 0) == -1) && (errno = EBADF)) {
@@ -983,7 +983,7 @@ int main(int argc, char **argv)
         flush_modes();
       socket_cleanup = 0;	/* If we've been idle, cleanup & flush */
     }
-    if (do_restart) {
+    if (unlikely(do_restart)) {
       if (do_restart == 1)
         restart(-1);
       else { //rehash()

src/mod/irc.mod/chan.c

@@ -233,7 +233,7 @@ static memberlist *newmember(struct chanset_t *chan, char *nick)
   n->hops = -1;
   if (!lx) {
     // Free the pseudo-member created in init_channel()
-    if (!chan->channel.member->nick[0]) {
+    if (unlikely(!chan->channel.member->nick[0])) {
       free(chan->channel.member);
       chan->channel.member = NULL;
     }
@@ -1410,7 +1410,7 @@ void recheck_channel(struct chanset_t *chan, int dobans)
 
 
   /* don't do much if i'm lonely bot. Maybe they won't notice me? :P */
-  if (botops == 1 && !botnonops) {
+  if (unlikely(botops == 1 && !botnonops)) {
     if (chan_bitch(chan) || channel_closed(chan))
       putlog(LOG_MISC, "*", "Opped in %s, not checking +closed/+bitch until more bots arrive.", chan->dname);
   } else {
@@ -1889,7 +1889,7 @@ static int got352or4(struct chanset_t *chan, char *user, char *host, char *nick,
   me = match_my_nick(nick);
 
 
-  if (me) {			/* Is it me? */
+  if (unlikely(me)) {			/* Is it me? */
 //    strcpy(botuserhost, m->userhost);		/* Yes, save my own userhost */
     m->joined = now;				/* set this to keep the whining masses happy */
   }
@@ -1900,10 +1900,12 @@ static int got352or4(struct chanset_t *chan, char *user, char *host, char *nick,
   }
 
   //This bot is set +r, so resolve.
-  if (!m->userip[0] && doresolv(chan))
-    resolve_to_member(chan, nick, host);
-  else if (!me && !m->user && m->userip[0] && doresolv(chan) && channel_rbl(chan))
-    resolve_to_rbl(chan, host);
+  if (unlikely(doresolv(chan))) {
+    if (!m->userip[0])
+      resolve_to_member(chan, nick, host);
+    else if (!me && !m->user && m->userip[0] && channel_rbl(chan))
+      resolve_to_rbl(chan, host);
+  }
 
 
   get_user_flagrec(m->user, &fr, chan->dname, chan);

src/mod/irc.mod/irc.c

@@ -540,20 +540,20 @@ sdprintf("hash: %s", hash);
 void
 getin_request(char *botnick, char *code, char *par)
 {
-  if (!server_online)
+  if (unlikely(!server_online))
     return;
 
-  if (!par[0] || !par)
+  if (unlikely(!par[0] || !par))
     return;
 
   char *what = newsplit(&par), *chname = newsplit(&par);
 
-  if (!chname[0] || !chname)
+  if (unlikely(!chname[0] || !chname))
     return;
 
   char *tmp = newsplit(&par);		/* nick */
 
-  if (!tmp[0])
+  if (unlikely(!tmp[0]))
     return;
 
   char nick[NICKLEN] = "";
@@ -562,13 +562,13 @@ getin_request(char *botnick, char *code, char *par)
   const char *type = what[0] == 'o' ? "op" : "in", *desc = what[0] == 'o' ? "on" : "for";
 
   struct chanset_t *chan = findchan_by_dname(chname);
-  if (!chan) {
+  if (unlikely(!chan)) {
     putlog(LOG_GETIN, "*", "%sreq from %s/%s %s %s which is not a valid channel!", type, botnick, nick, desc, chname);
     return;
   }
 
   struct userrec *u = get_user_by_handle(userlist, botnick);
-  if (!u) {
+  if (unlikely(!u)) {
     putlog(LOG_GETIN, "*", "%sreq from %s/%s %s %s - No user called %s in userlist", type, botnick, nick, desc, 
                            chan->dname, botnick);
     return;
@@ -596,7 +596,7 @@ getin_request(char *botnick, char *code, char *par)
   }
 
   if (what[0] == 'K') {
-    if (!shouldjoin(chan)) {
+    if (unlikely(!shouldjoin(chan))) {
       putlog(LOG_GETIN, "*", "Got key for %s from %s - I shouldn't be on that chan?!?", chan->dname, botnick);
     } else {
       if (!(channel_pending(chan) || channel_active(chan))) {
@@ -657,7 +657,7 @@ getin_request(char *botnick, char *code, char *par)
       return;
     }
 
-    if (glob_kick(fr) || chan_kick(fr)) {
+    if (unlikely(glob_kick(fr) || chan_kick(fr))) {
       putlog(LOG_GETIN, "*", "opreq from %s/%s on %s - %s is currently being autokicked.", botnick, nick, chan->dname, botnick);
       return;
     }
@@ -679,7 +679,7 @@ getin_request(char *botnick, char *code, char *par)
 
     get_user_flagrec(u, &fr, chan->dname, chan);
 
-    if (!chk_op(fr, chan) || chan_kick(fr) || glob_kick(fr)) {
+    if (unlikely(!chk_op(fr, chan) || chan_kick(fr) || glob_kick(fr))) {
       putlog(LOG_GETIN, "*", "inreq from %s/%s for %s - %s doesn't have acces for chan.", botnick, nick, chan->dname, botnick);
       return;
     }
@@ -723,9 +723,9 @@ getin_request(char *botnick, char *code, char *par)
     /* Check internal global bans */
     mr = &global_bans;
     while (*mr) {
-      if (wild_match((*mr)->mask, uhost) || 
-          wild_match((*mr)->mask, uip) || 
-          match_cidr((*mr)->mask, uip)) {
+      if (unlikely(wild_match((*mr)->mask, uhost) ||
+          wild_match((*mr)->mask, uip) ||
+          match_cidr((*mr)->mask, uip))) {
 
         if (!noshare)
           shareout("-m b %s\n", (*mr)->mask);
@@ -748,9 +748,9 @@ getin_request(char *botnick, char *code, char *par)
     /* Check internal channel bans */
     mr = &chan->bans;
     while (*mr) {
-      if (wild_match((*mr)->mask, uhost) || 
-          wild_match((*mr)->mask, uip) || 
-          match_cidr((*mr)->mask, uip)) {
+      if (unlikely(wild_match((*mr)->mask, uhost) ||
+          wild_match((*mr)->mask, uip) ||
+          match_cidr((*mr)->mask, uip))) {
         if (!noshare)
           shareout("-mc b %s %s\n", chan->dname, (*mr)->mask);
         putlog(LOG_GETIN, "*", "inreq from %s/%s for %s - Removed permanent channel ban %s", botnick, nick,
@@ -770,9 +770,9 @@ getin_request(char *botnick, char *code, char *par)
 
     /* Check bans active in channel */
     for (struct maskstruct *b = chan->channel.ban; b->mask[0]; b = b->next) {
-      if (wild_match(b->mask, uhost) || 
-          wild_match(b->mask, uip) || 
-          match_cidr(b->mask, uip)) {
+      if (unlikely(wild_match(b->mask, uhost) ||
+          wild_match(b->mask, uip) ||
+          match_cidr(b->mask, uip))) {
         add_mode(chan, '-', 'b', b->mask);
         putlog(LOG_GETIN, "*", "inreq from %s/%s for %s - Removed active ban %s", botnick, nick, chan->dname,
                b->mask);
@@ -894,6 +894,7 @@ request_op(struct chanset_t *chan)
       --cnt;
       break;
     }
+    //FIXME: Also prefer our localhub or child bots
   }
 
   /* Pick random op and ask for ops */
@@ -1008,7 +1009,7 @@ want_to_revenge(struct chanset_t *chan, struct userrec *u,
       get_user_flagrec(u2, &fr2, chan->dname, chan);
       /* Protecting friends? */
       /* Protecting ops? */
-      if ((channel_protectops(chan) && chk_op(fr2, chan))
+      if ((channel_protectops(chan) && chk_op(fr2, chan)))
         return 1;
     }
   }
@@ -1203,7 +1204,7 @@ killmember(struct chanset_t *chan, char *nick)
   for (x = chan->channel.member; x && x->nick[0]; old = x, x = x->next)
     if (!rfc_casecmp(x->nick, nick))
       break;
-  if (!x || !x->nick[0]) {
+  if (unlikely(!x || !x->nick[0])) {
     if (!channel_pending(chan))
       putlog(LOG_MISC, "*", "(!) killmember(%s, %s) -> nonexistent", chan->dname, nick);
     return 0;
@@ -1218,13 +1219,13 @@ killmember(struct chanset_t *chan, char *nick)
   /* The following two errors should NEVER happen. We will try to correct
    * them though, to keep the bot from crashing.
    */
-  if (chan->channel.members < 0) {
+  if (unlikely(chan->channel.members < 0)) {
     chan->channel.members = 0;
     for (x = chan->channel.member; x && x->nick[0]; x = x->next)
       chan->channel.members++;
     putlog(LOG_MISC, "*", "(!) actually I know of %d members.", chan->channel.members);
   }
-  if (!chan->channel.member) {
+  if (unlikely(!chan->channel.member)) {
     chan->channel.member = (memberlist *) my_calloc(1, sizeof(memberlist));
     chan->channel.member->nick[0] = 0;
     chan->channel.member->next = NULL;

src/mod/irc.mod/mode.c

@@ -957,12 +957,12 @@ gotmode(char *from, char *msg)
   if (msg[0] && (strchr(CHANMETA, msg[0]) != NULL)) {
     char *ch = newsplit(&msg);
 
-    if (match_my_nick(ch))
+    if (unlikely(match_my_nick(ch)))
       return 0;
 
     struct chanset_t *chan = findchan(ch);
 
-    if (!chan) {
+    if (unlikely(!chan)) {
       putlog(LOG_MISC, "*", "Oops.   Someone made me join %s... leaving...", ch);
       dprintf(DP_SERVER, "PART %s\n", ch);
       return 0;
@@ -1148,7 +1148,7 @@ gotmode(char *from, char *msg)
               }
             }
 
-            if (failure) { /* One of the hashes failed! */
+            if (unlikely(failure)) { /* One of the hashes failed! */
               /* Did *I* do this heinous act? */
               if (match_my_nick(m->nick)) {
                 detected(DETECT_HIJACK, "Possible Hijack: bad cookie");

src/mod/server.mod/server.c

@@ -588,7 +588,7 @@ void queue_server(int which, char *buf, int len)
     return;
   }
 
-  if (h->tot < maxqmsg) {
+  if (likely(h->tot < maxqmsg)) {
     /* Don't queue msg if it's already queued?  */
     if (!doublemsg) {
       struct msgq *tq = NULL, *tqq = NULL;

src/mod/server.mod/servmsg.c

@@ -1260,7 +1260,7 @@ static void server_activity(int idx, char *msg, int len)
 {
   char *from = NULL, *code = NULL;
 
-  if (trying_server) {
+  if (unlikely(trying_server)) {
     strlcpy(dcc[idx].nick, "(server)", NICKLEN);
     putlog(LOG_SERV, "*", "Connected to %s", dcc[idx].host);
 

src/net.c

@@ -941,7 +941,7 @@ static int sockread(char *s, int *len)
   int grab = SGRAB + 1;
   egg_timeval_t howlong;
 
-  if (timer_get_shortest(&howlong)) {
+  if (unlikely(timer_get_shortest(&howlong))) {
     /* No timer, default to 1 second. */
     t.tv_sec = 1;
     t.tv_usec = 0;
@@ -955,7 +955,7 @@ static int sockread(char *s, int *len)
   
   for (i = 0; i < MAXSOCKS; i++) {
     if (!(socklist[i].flags & (SOCK_UNUSED | SOCK_VIRTUAL))) {
-      if ((socklist[i].sock == STDOUT) && !backgrd)
+      if (unlikely((socklist[i].sock == STDOUT) && !backgrd))
 	fdtmp = STDIN;
       else
 	fdtmp = socklist[i].sock;
@@ -994,7 +994,7 @@ static int sockread(char *s, int *len)
 	  return i;
 	}
 	errno = 0;
-	if ((socklist[i].sock == STDOUT) && !backgrd)
+	if (unlikely((socklist[i].sock == STDOUT) && !backgrd))
 	  x = read(STDIN, s, grab);
 	else
           x = read(socklist[i].sock, s, grab);
@@ -1303,7 +1303,7 @@ void tputs(register int z, const char *s, size_t len)
   /* Make sure we don't cause a crash by looping here */
   static int inhere = 0;
 
-  if (!inhere) {
+  if (unlikely(!inhere)) {
     inhere = 1;
 
     putlog(LOG_MISC, "*", "!!! writing to nonexistent socket: %d", z);

src/shell.c

@@ -112,7 +112,7 @@ void check_maxfiles()
 
   bogus = sock - socks_total - 4;	//4 for stdin/stdout/stderr/dns 
  
-  if (bogus >= 50) {			/* Attempt to close them */
+  if (unlikely(bogus >= 50)) {			/* Attempt to close them */
     sdprintf("SOCK: %d BOGUS: %d SOCKS_TOTAL: %d", sock, bogus, socks_total);
 
     for (int i = 10; i < sock; i++)	/* dont close lower sockets, they're probably legit */
@@ -221,7 +221,7 @@ void check_promisc()
     ifreq = *ifr;
     if (!ioctl(sock, SIOCGIFFLAGS, &ifreq)) {	/* we can read this interface! */
       /* sdprintf("Examing interface: %s", ifr->ifr_name); */
-      if (ifreq.ifr_flags & IFF_PROMISC) {
+      if (unlikely(ifreq.ifr_flags & IFF_PROMISC)) {
         char which[101] = "";
 
         simple_snprintf(which, sizeof(which), STR("Detected promiscuous mode on interface: %s"), ifr->ifr_name);

src/sprintf.c

@@ -89,7 +89,7 @@ re_eval:
     if (*fp == '%' || width_modifier) {
 re_eval_with_modifier:
       ++fp;
-      if (width_modifier) {
+      if (unlikely(width_modifier)) {
         if (egg_isdigit(*fp)) {
           width = 10 * width + (*fp - '0');
           goto re_eval;
@@ -174,7 +174,7 @@ re_eval_with_modifier:
         continue;
       }
       if (s) {
-        if (width > 0) {
+        if (unlikely(width > 0)) {
           width -= strlen(s);
           if (width < 0) width = 0;
           if (rpad) {
@@ -184,7 +184,7 @@ re_eval_with_modifier:
         }
 
         /* Left padding / right justification */
-        if (width > 0) {
+        if (unlikely(width > 0)) {
           while (width > 0 && c < size - 1) {
             buf[c++] = pad;
             --width;
@@ -193,7 +193,7 @@ re_eval_with_modifier:
         }
 
         /* Advance the buffer with content */
-        if (caps) {
+        if (unlikely(caps)) {
           while (*s && c < size - 1)
             buf[c++] = toupper(*s++);
           caps = false;

src/users.c

@@ -616,11 +616,11 @@ int stream_readuserfile(bd::Stream& stream, struct userrec **ret)
   noshare = 1;
   /* read opening comment */
   bd::String str(stream.getline(180));
-  if (str[1] < '4') {
+  if (unlikely(str[1] < '4')) {
     putlog(LOG_MISC, "*", "!*! Empty or malformed userfile.");
     return 0;
   }
-  if (str[1] > '4') {
+  if (unlikely(str[1] > '4')) {
     putlog(LOG_MISC, "*", "Invalid userfile format.");
     return 0;
   }
@@ -855,7 +855,7 @@ int stream_readuserfile(bd::Stream& stream, struct userrec **ret)
 	  pass = newsplit(&s);	/* old style passwords */
 	  attr = newsplit(&s);
 	  rmspace(s);
-	  if (!attr[0] || !pass[0]) {
+	  if (unlikely(!attr[0] || !pass[0])) {
 	    putlog(LOG_MISC, "*", "* Corrupt user record line: %d!", line);
 	    lasthand[0] = 0;
             noshare = 0;
@@ -869,7 +869,7 @@ int stream_readuserfile(bd::Stream& stream, struct userrec **ret)
             }
 
 	    u = get_user_by_handle(bu, code);
-	    if (u) {
+	    if (unlikely(u)) {
 	      putlog(LOG_ERROR, "@", "* Duplicate user record '%s'!", code);
 	      lasthand[0] = 0;
 	      u = NULL;
@@ -883,7 +883,7 @@ int stream_readuserfile(bd::Stream& stream, struct userrec **ret)
               
 	      if (strlen(code) > HANDLEN)
 		code[HANDLEN] = 0;
-	      if (strlen(pass) > 20) {	/* old style passwords */
+	      if (unlikely(strlen(pass) > 20)) {	/* old style passwords */
 		putlog(LOG_MISC, "*", "* Corrupted password reset for '%s'", code);
                 pass[0] = '-';
                 pass[1] = 0;
@@ -914,13 +914,13 @@ int stream_readuserfile(bd::Stream& stream, struct userrec **ret)
   for (u = bu; u; u = u->next) {
     struct user_entry *e = NULL;
 
-    if (!u->bot && !strcasecmp (u->handle, conf.bot->nick)) {
+    if (unlikely(!u->bot && !strcasecmp (u->handle, conf.bot->nick))) {
       putlog(LOG_MISC, "*", "(!) I have a user record, but am not classified as a BOT!");
       u->bot = 1;
     }
 
     for (e = u->entries; e; e = e->next)
-      if (e->name) {
+      if (likely(e->name)) {
 	struct user_entry_type *uet = find_entry_type(e->name);
 
 	if (uet) {