names: refactor to use multiline

[solanum.git] / modules / m_names.c

diff --git a/modules/m_names.c b/modules/m_names.c
index 766fac03a43edd5520ff4e65c1d422c4555002f7..09c63fe07d730a6544795d430248796671c247d4 100644 (file)
--- a/modules/m_names.c
+++ b/modules/m_names.c
@@ -25,7 +25,6 @@
 #include "stdinc.h"
 #include "channel.h"
 #include "client.h"
-#include "common.h"
 #include "hash.h"
 #include "match.h"
 #include "ircd.h"
@@ -36,8 +35,11 @@
 #include "msg.h"
 #include "parse.h"
 #include "modules.h"
+#include "s_newconf.h"
 
-static int m_names(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
+static const char names_desc[] = "Provides the NAMES command to view users on a channel";
+
+static void m_names(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
 
 struct Message names_msgtab = {
        "NAMES", 0, 0, 0, 0,
@@ -46,8 +48,6 @@ struct Message names_msgtab = {
 
 mapi_clist_av1 names_clist[] = { &names_msgtab, NULL };
 
-static const char names_desc[] = "Provides the NAMES command to view users on a channel";
-
 DECLARE_MODULE_AV2(names, NULL, NULL, names_clist, NULL, NULL, NULL, NULL, names_desc);
 
 static void names_global(struct Client *source_p);
@@ -60,7 +60,7 @@ static void names_global(struct Client *source_p);
  * m_names
  *      parv[1] = channel
  */
-static int
+static void
 m_names(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
        static time_t last_used = 0;
@@ -78,7 +78,7 @@ m_names(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_
                        sendto_one_numeric(source_p, ERR_BADCHANNAME,
                                           form_str(ERR_BADCHANNAME),
                                           (unsigned char *) p);
-                       return 0;
+                       return;
                }
 
                if((chptr = find_channel(p)) != NULL)
@@ -89,7 +89,7 @@ m_names(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_
        }
        else
        {
-               if(!IsOper(source_p))
+               if(!IsOperGeneral(source_p))
                {
                        if((last_used + ConfigFileEntry.pace_wait) > rb_current_time())
                        {
@@ -97,7 +97,7 @@ m_names(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_
                                           me.name, source_p->name, "NAMES");
                                sendto_one(source_p, form_str(RPL_ENDOFNAMES),
                                           me.name, source_p->name, "*");
-                               return 0;
+                               return;
                        }
                        else
                                last_used = rb_current_time();
@@ -107,8 +107,6 @@ m_names(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_
                sendto_one(source_p, form_str(RPL_ENDOFNAMES),
                           me.name, source_p->name, "*");
        }
-
-       return 0;
 }
 
 /*
@@ -121,16 +119,11 @@ m_names(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_
 static void
 names_global(struct Client *source_p)
 {
-       int mlen;
-       int tlen;
-       int cur_len;
-       int dont_show = NO;
+       bool dont_show = false;
        rb_dlink_node *lp, *ptr;
        struct Client *target_p;
        struct Channel *chptr = NULL;
        struct membership *msptr;
-       char buf[BUFSIZE];
-       char *t;
 
        /* first do all visible channels */
        RB_DLINK_FOREACH(ptr, global_channel_list.head)
@@ -138,15 +131,18 @@ names_global(struct Client *source_p)
                chptr = ptr->data;
                channel_member_names(chptr, source_p, 0);
        }
-       cur_len = mlen = sprintf(buf, form_str(RPL_NAMREPLY),
-                                   me.name, source_p->name, "*", "*");
-       t = buf + mlen;
+
+       send_multiline_init(source_p, " ", form_str(RPL_NAMREPLY),
+                                   me.name,
+                                   source_p->name,
+                                   "*",
+                                   "*");
 
        /* Second, do all clients in one big sweep */
        RB_DLINK_FOREACH(ptr, global_client_list.head)
        {
                target_p = ptr->data;
-               dont_show = NO;
+               dont_show = false;
 
                if(!IsPerson(target_p) || IsInvisible(target_p))
                        continue;
@@ -166,7 +162,7 @@ names_global(struct Client *source_p)
                        if(PubChannel(chptr) || IsMember(source_p, chptr) ||
                           SecretChannel(chptr))
                        {
-                               dont_show = YES;
+                               dont_show = true;
                                break;
                        }
                }
@@ -174,18 +170,18 @@ names_global(struct Client *source_p)
                if(dont_show)
                        continue;
 
-               if((cur_len + NICKLEN + 2) > (BUFSIZE - 3))
+               if (IsCapable(source_p, CLICAP_USERHOST_IN_NAMES))
                {
-                       sendto_one(source_p, "%s", buf);
-                       cur_len = mlen;
-                       t = buf + mlen;
+                       send_multiline_item(source_p, "%s!%s@%s",
+                                       target_p->name,
+                                       target_p->username,
+                                       target_p->host);
+               }
+               else
+               {
+                       send_multiline_item(source_p, "%s", target_p->name);
                }
-
-               tlen = sprintf(t, "%s ", target_p->name);
-               cur_len += tlen;
-               t += tlen;
        }
 
-       if(cur_len > mlen)
-               sendto_one(source_p, "%s", buf);
+       send_multiline_fini(source_p, NULL);
 }