Some more text fixes and improvements (#1618).

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

diff --git a/src/modules/m_setidle.cpp b/src/modules/m_setidle.cpp
index 4a15fd0d5a050fff9c33f6694ae94d7909aba858..eaae082499d0c1dfcbccb018cdd197b6fd534956 100644 (file)
--- a/src/modules/m_setidle.cpp
+++ b/src/modules/m_setidle.cpp
@@ -21,6 +21,13 @@
 
 #include "inspircd.h"
 
+enum
+{
+       // InspIRCd-specific.
+       ERR_INVALIDIDLETIME = 948,
+       RPL_IDLETIMESET = 944
+};
+
 /** Handle /SETIDLE
  */
 class CommandSetidle : public SplitCommand
@@ -31,12 +38,12 @@ class CommandSetidle : public SplitCommand
                flags_needed = 'o'; syntax = "<duration>";
        }
 
-       CmdResult HandleLocal(const std::vector<std::string>& parameters, LocalUser* user)
+       CmdResult HandleLocal(LocalUser* user, const Params& parameters) CXX11_OVERRIDE
        {
-               int idle = InspIRCd::Duration(parameters[0]);
-               if (idle < 1)
+               unsigned long idle;
+               if (!InspIRCd::Duration(parameters[0], idle))
                {
-                       user->WriteNumeric(948, "Invalid idle time.");
+                       user->WriteNumeric(ERR_INVALIDIDLETIME, "Invalid idle time.");
                        return CMD_FAILURE;
                }
                user->idle_lastmsg = (ServerInstance->Time() - idle);
@@ -44,7 +51,7 @@ class CommandSetidle : public SplitCommand
                if (user->signon > user->idle_lastmsg)
                        user->signon = user->idle_lastmsg;
                ServerInstance->SNO->WriteToSnoMask('a', user->nick+" used SETIDLE to set their idle time to "+ConvToStr(idle)+" seconds");
-               user->WriteNumeric(944, "Idle time set.");
+               user->WriteNumeric(RPL_IDLETIMESET, "Idle time set.");
 
                return CMD_SUCCESS;
        }
@@ -62,7 +69,7 @@ class ModuleSetIdle : public Module
 
        Version GetVersion() CXX11_OVERRIDE
        {
-               return Version("Allows opers to set their idle time", VF_VENDOR);
+               return Version("Provides the SETIDLE command, allows opers to set their idle time", VF_VENDOR);
        }
 };