Message handlers should return void.

[solanum.git] / modules / m_cap.c

diff --git a/modules/m_cap.c b/modules/m_cap.c
index b2e4a78a179b1abbbc9ed0910466d938b8dd2388..726fbe4a894e55db5d85f957aefbb2e322293cb2 100644 (file)
--- a/modules/m_cap.c
+++ b/modules/m_cap.c
@@ -44,9 +44,11 @@
 #include "s_conf.h"
 #include "hash.h"
 
+static const char cap_desc[] = "Provides the commands used for client capability negotiation";
+
 typedef int (*bqcmp)(const void *, const void *);
 
-static int m_cap(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
+static void m_cap(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
 
 struct Message cap_msgtab = {
        "CAP", 0, 0, 0, 0,
@@ -54,7 +56,8 @@ struct Message cap_msgtab = {
 };
 
 mapi_clist_av1 cap_clist[] = { &cap_msgtab, NULL };
-DECLARE_MODULE_AV2(cap, NULL, NULL, cap_clist, NULL, NULL, NULL, NULL, NULL);
+
+DECLARE_MODULE_AV2(cap, NULL, NULL, cap_clist, NULL, NULL, NULL, NULL, cap_desc);
 
 #define IsCapableEntry(c, e)           IsCapable(c, 1 << (e)->value)
 #define HasCapabilityFlag(c, f)                (c->ownerdata != NULL && (((struct ClientCapability *)c->ownerdata)->flags & (f)) == f)
@@ -152,20 +155,16 @@ clicap_generate(struct Client *source_p, const char *subcmd, int flags, int clea
 {
        char buf[BUFSIZE] = { 0 };
        char capbuf[BUFSIZE] = { 0 };
-       char *p;
        int buflen = 0;
        int curlen, mlen;
        struct CapabilityEntry *entry;
        struct DictionaryIter iter;
 
-       mlen = sprintf(buf, ":%s CAP %s %s",
+       mlen = snprintf(buf, sizeof buf, ":%s CAP %s %s",
                        me.name,
                        EmptyString(source_p->name) ? "*" : source_p->name,
                        subcmd);
 
-       p = capbuf;
-       buflen = mlen;
-
        /* shortcut, nothing to do */
        if(flags == -1)
        {
@@ -199,43 +198,25 @@ clicap_generate(struct Client *source_p, const char *subcmd, int flags, int clea
                        caplen += strlen(data) + 1;
 
                /* \r\n\0, possible "-~=", space, " *" */
-               if(buflen + caplen >= BUFSIZE - 10)
+               if(buflen + mlen >= BUFSIZE - 10)
                {
                        /* remove our trailing space -- if buflen == mlen
                         * here, we didnt even succeed in adding one.
                         */
-                       if(buflen != mlen)
-                               *(p - 1) = '\0';
-                       else
-                               *p = '\0';
+                       capbuf[buflen] = '\0';
 
                        sendto_one(source_p, "%s * :%s", buf, capbuf);
 
-                       p = capbuf;
                        buflen = mlen;
                        memset(capbuf, 0, sizeof capbuf);
                }
 
-               if(clear)
-               {
-                       *p++ = '-';
-                       buflen++;
-               }
-
-               if (data == NULL)
-                       curlen = sprintf(p, "%s ", entry->cap);
-               else
-                       curlen = sprintf(p, "%s=%s ", entry->cap, data);
-
-               p += curlen;
-               buflen += curlen;
+               buflen = rb_snprintf_append(capbuf, sizeof capbuf, "%s%s%s%s ",
+                               clear ? "-" : "", entry->cap, data != NULL ? "=" : "", data != NULL ? data : "");
        }
 
        /* remove trailing space */
-       if(buflen != mlen)
-               *(p - 1) = '\0';
-       else
-               *p = '\0';
+       capbuf[buflen] = '\0';
 
        sendto_one(source_p, "%s :%s", buf, capbuf);
 }
@@ -436,7 +417,7 @@ clicap_cmd_search(const char *command, struct clicap_cmd *entry)
        return irccmp(command, entry->cmd);
 }
 
-static int
+static void
 m_cap(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
        struct clicap_cmd *cmd;
@@ -448,9 +429,8 @@ m_cap(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p,
                sendto_one(source_p, form_str(ERR_INVALIDCAPCMD),
                                me.name, EmptyString(source_p->name) ? "*" : source_p->name,
                                parv[1]);
-               return 0;
+               return;
        }
 
        (cmd->func)(source_p, parv[2]);
-       return 0;
 }