Mode +b stuff, probably wont work yet

[user/henk/code/inspircd.git] / src / modules / m_helpop.cpp

diff --git a/src/modules/m_helpop.cpp b/src/modules/m_helpop.cpp
index b3b3d113a3e55f055b5e5a2d35725551259f1c7d..b6c723ca79e1cd126a1054d90cc1d39322497bf5 100644 (file)
--- a/src/modules/m_helpop.cpp
+++ b/src/modules/m_helpop.cpp
@@ -22,9 +22,8 @@ using namespace std;
 #include "helperfuncs.h"
 
 // Global Vars
-/* XXX - should all this be marked static? clear the global namespace, etc. */
-ConfigReader *helpop;
-Server *Srv;
+static ConfigReader *helpop;
+static Server *Srv;
 
 void handle_helpop(char**, int, userrec*);
 bool do_helpop(char**, int, userrec*);
@@ -35,7 +34,7 @@ void sendtohelpop(userrec*, int, char**);
 class cmd_helpop : public command_t
 {
  public:
-        cmd_helpop () : command_t("HELPOP",0,0)
+        cmd_helpop () : command_t("HELPOP",0,1)
         {
                 this->source = "m_helpop.so";
         }
@@ -106,10 +105,9 @@ bool do_helpop(char **parameters, int pcnt, userrec *src)
        char *search;
        std::string output = " "; // a fix bought to you by brain :p
        char a[MAXBUF];
-       char lower[MAXBUF];
        int nlines = 0;
 
-       if (!parameters)
+       if (!pcnt)
        {
                search = "start";
        }
@@ -123,16 +121,12 @@ bool do_helpop(char **parameters, int pcnt, userrec *src)
                search++;
        }
 
-       /* XXX - don't we have an strtolower()? if not, might pay to add one.. that works on char *, preferably.. */
-       strlcpy(lower, search, MAXBUF);
-       for (unsigned int t = 0; t < strlen(lower); t++)
-               lower[t] = tolower(lower[t]);
-
+       strlower(search);
 
        for (int i = 1; output != ""; i++)
        {
                snprintf(a,MAXBUF,"line%d",i);
-               output = helpop->ReadValue(lower, a, 0);
+               output = helpop->ReadValue(search, a, 0);
                if (output != "")
                {
                        Srv->SendTo(NULL,src,"290 "+std::string(src->nick)+" :"+output);
@@ -166,7 +160,7 @@ class HelpopException : public ModuleException
        std::string err;
  public:
        HelpopException(std::string message) : err(message) { }
-       virtual char* GetReason() { return (char*)err.c_str(); }
+       virtual const char* GetReason() { return err.c_str(); }
 };
 
 class ModuleHelpop : public Module
@@ -220,11 +214,11 @@ class ModuleHelpop : public Module
                        List[I_OnRehash] = List[I_OnExtendedMode] = List[I_OnWhois] = 1;
                }
 
-               virtual void OnRehash(std::string parameter)
+               virtual void OnRehash(const std::string &parameter)
                {
-                       delete conf;
+                       DELETE(conf);
                        if (helpop)
-                               delete helpop;
+                               DELETE(helpop);
 
                        ReadConfig();
                }
@@ -248,8 +242,8 @@ class ModuleHelpop : public Module
 
                virtual ~ModuleHelpop()
                {
-                       delete conf;
-                       delete helpop;
+                       DELETE(conf);
+                       DELETE(helpop);
                }
        
                virtual Version GetVersion()