m_spanningtree Remove redundant params.size() checks

TreeSocket::ProcessConnectedLine() and CommandParser::CallHandler() only call the handler when there are enough parameters

diff --git a/src/modules/m_spanningtree/fjoin.cpp b/src/modules/m_spanningtree/fjoin.cpp
index 929ace474f1296271c18c1dd94eb3b098b33402a..71d837b2840b4bf0b150029ac935fd12a484115e 100644 (file)
--- a/src/modules/m_spanningtree/fjoin.cpp
+++ b/src/modules/m_spanningtree/fjoin.cpp
@@ -55,8 +55,6 @@ CmdResult CommandFJoin::Handle(const std::vector<std::string>& params, User *src
         * losing side, so only its own modes get applied. Life is simple for those
         * who succeed at internets. :-)
         */
-       if (params.size() < 3)
-               return CMD_INVALID;
 
        irc::modestacker modestack(true);                       /* Modes to apply from the users in the user list */
        User* who = NULL;                                               /* User we are currently checking */

diff --git a/src/modules/m_spanningtree/uid.cpp b/src/modules/m_spanningtree/uid.cpp
index 4a73eb80c2d69c1d4a070c4d1014401d3808180f..f33cb038d772f8554f1289f943b7e1d53e53743c 100644 (file)
--- a/src/modules/m_spanningtree/uid.cpp
+++ b/src/modules/m_spanningtree/uid.cpp
@@ -35,9 +35,6 @@ CmdResult CommandUID::Handle(const parameterlist &params, User* serversrc)
         *      0    1    2    3    4    5        6        7     8        9       (n-1)
         * UID uuid age nick host dhost ident ip.string signon +modes (modepara) :gecos
         */
-       if (params.size() < 10)
-               return CMD_INVALID;
-
        time_t age_t = ConvToInt(params[1]);
        time_t signon = ConvToInt(params[7]);
        std::string empty;
@@ -163,8 +160,6 @@ CmdResult CommandUID::Handle(const parameterlist &params, User* serversrc)
 
 CmdResult CommandFHost::Handle(const parameterlist &params, User* src)
 {
-       if (params.size() < 1)
-               return CMD_INVALID;
        if (IS_SERVER(src))
                return CMD_FAILURE;
        src->ChangeDisplayedHost(params[0].c_str());
@@ -173,8 +168,6 @@ CmdResult CommandFHost::Handle(const parameterlist &params, User* src)
 
 CmdResult CommandFIdent::Handle(const parameterlist &params, User* src)
 {
-       if (params.size() < 1)
-               return CMD_INVALID;
        if (IS_SERVER(src))
                return CMD_FAILURE;
        src->ChangeIdent(params[0].c_str());
@@ -183,8 +176,6 @@ CmdResult CommandFIdent::Handle(const parameterlist &params, User* src)
 
 CmdResult CommandFName::Handle(const parameterlist &params, User* src)
 {
-       if (params.size() < 1)
-               return CMD_INVALID;
        if (IS_SERVER(src))
                return CMD_FAILURE;
        src->ChangeName(params[0].c_str());