Get rid of a bunch of memory-wasting C-style strings

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

diff --git a/src/modules/m_maphide.cpp b/src/modules/m_maphide.cpp
index 7f96f18dd30619324a8d133bd1bd89e4ff5e16ae..4c9d0591f38905c5cf132560db6e488ea5ab378b 100644 (file)
--- a/src/modules/m_maphide.cpp
+++ b/src/modules/m_maphide.cpp
@@ -19,30 +19,28 @@ class ModuleMapHide : public Module
 {
        std::string url;
  public:
-       ModuleMapHide(InspIRCd* Me)
-               : Module(Me)
-       {
-               // Create a new command
+       ModuleMapHide()
+                       {
                ServerInstance->Modules->Attach(I_OnPreCommand, this);
                ServerInstance->Modules->Attach(I_OnRehash, this);
-               OnRehash(NULL, "");
+               OnRehash(NULL);
        }
 
-       void OnRehash(User* user, const std::string &parameter)
+       void OnRehash(User* user)
        {
-               ConfigReader MyConf(ServerInstance);
+               ConfigReader MyConf;
                url = MyConf.ReadValue("security", "maphide", 0);
        }
 
-       int OnPreCommand(std::string &command, std::vector<std::string> &parameters, User *user, bool validated, const std::string &original_line)
+       ModResult OnPreCommand(std::string &command, std::vector<std::string> &parameters, User *user, bool validated, const std::string &original_line)
        {
                if (!IS_OPER(user) && !url.empty() && (command == "MAP" || command == "LINKS"))
                {
                        user->WriteServ("NOTICE %s :/%s has been disabled; visit %s", user->nick.c_str(), command.c_str(), url.c_str());
-                       return 1;
+                       return MOD_RES_DENY;
                }
                else
-                       return 0;
+                       return MOD_RES_PASSTHRU;
        }
 
        virtual ~ModuleMapHide()
@@ -51,7 +49,7 @@ class ModuleMapHide : public Module
 
        virtual Version GetVersion()
        {
-               return Version("$Id$", VF_VENDOR, API_VERSION);
+               return Version("Hide /MAP and /LINKS in the same form as ircu (mostly useless)", VF_VENDOR, API_VERSION);
        }
 };