Message handlers should return void.

[solanum.git] / modules / m_rehash.c

diff --git a/modules/m_rehash.c b/modules/m_rehash.c
index 3caef467e32235aa0fb3c39681cd56196856c282..ee98377255d2788682d932fb068c8aa5b0d18552 100644 (file)
--- a/modules/m_rehash.c
+++ b/modules/m_rehash.c
@@ -20,8 +20,6 @@
  *  along with this program; if not, write to the Free Software
  *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307
  *  USA
- *
- *  $Id: m_rehash.c 3161 2007-01-25 07:23:01Z nenolod $
  */
 
 #include "stdinc.h"
@@ -43,11 +41,14 @@
 #include "reject.h"
 #include "hash.h"
 #include "cache.h"
-#include "irc_radixtree.h"
+#include "rb_radixtree.h"
 #include "sslproc.h"
 
-static int mo_rehash(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
-static int me_rehash(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
+static const char rehash_desc[] =
+       "Provides the REHASH command to reload configuration and other files";
+
+static void mo_rehash(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
+static void me_rehash(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
 
 struct Message rehash_msgtab = {
        "REHASH", 0, 0, 0, 0,
@@ -55,7 +56,8 @@ struct Message rehash_msgtab = {
 };
 
 mapi_clist_av1 rehash_clist[] = { &rehash_msgtab, NULL };
-DECLARE_MODULE_AV1(rehash, NULL, NULL, rehash_clist, NULL, NULL, "$Revision: 3161 $");
+
+DECLARE_MODULE_AV2(rehash, NULL, NULL, rehash_clist, NULL, NULL, NULL, NULL, rehash_desc);
 
 struct hash_commands
 {
@@ -196,7 +198,7 @@ static void
 rehash_tresvs(struct Client *source_p)
 {
        struct ConfItem *aconf;
-       struct irc_radixtree_iteration_state iter;
+       struct rb_radixtree_iteration_state iter;
        rb_dlink_node *ptr;
        rb_dlink_node *next_ptr;
        int i;
@@ -206,12 +208,12 @@ rehash_tresvs(struct Client *source_p)
        if (!MyConnect(source_p))
                remote_rehash_oper_p = source_p;
 
-       IRC_RADIXTREE_FOREACH(aconf, &iter, resv_tree)
+       RB_RADIXTREE_FOREACH(aconf, &iter, resv_tree)
        {
                if(!aconf->hold || aconf->lifetime)
                        continue;
 
-               irc_radixtree_delete(resv_tree, aconf->host);
+               rb_radixtree_delete(resv_tree, aconf->host);
                free_conf(aconf);
        }
 
@@ -355,7 +357,7 @@ do_rehash(struct Client *source_p, const char *type)
  * parv[1] = rehash type or destination
  * parv[2] = destination
  */
-static int
+static void
 mo_rehash(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
        const char *type = NULL, *target_server = NULL;
@@ -364,7 +366,7 @@ mo_rehash(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *sourc
        {
                sendto_one(source_p, form_str(ERR_NOPRIVS),
                           me.name, source_p->name, "rehash");
-               return 0;
+               return;
        }
 
        if (parc > 2)
@@ -382,33 +384,29 @@ mo_rehash(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *sourc
                {
                        sendto_one(source_p, form_str(ERR_NOPRIVS),
                                me.name, source_p->name, "remoteban");
-                       return 0;
+                       return;
                }
                sendto_match_servs(source_p, target_server,
                                CAP_ENCAP, NOCAPS,
                                "ENCAP %s REHASH %s",
                                target_server, type != NULL ? type : "");
                if (match(target_server, me.name) == 0)
-                       return 0;
+                       return;
        }
 
        do_rehash(source_p, type);
-
-       return 0;
 }
 
-static int
+static void
 me_rehash(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
 
        if (!IsPerson(source_p))
-               return 0;
+               return;
 
        if (!find_shared_conf(source_p->username, source_p->host,
                                source_p->servptr->name, SHARED_REHASH))
-               return 0;
+               return;
 
        do_rehash(source_p, parc > 1 ? parv[1] : NULL);
-
-       return 0;
 }