More (last of extra)

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

diff --git a/src/modules/m_opermodes.cpp b/src/modules/m_opermodes.cpp
index a8aa615cf0f5cab47eb19149c3599eeb7dedc6c7..e358492bca871adead7455298cbe5ce3be63d174 100644 (file)
--- a/src/modules/m_opermodes.cpp
+++ b/src/modules/m_opermodes.cpp
@@ -65,14 +65,14 @@ class ModuleModesOnOper : public Module
                        if (!strcmp(typen.c_str(),user->oper))
                        {
                                std::string ThisOpersModes = Conf->ReadValue("type","modes",j);
-                               char first = *(ThisOpersModes.c_str());
-                               if ((first != '+') && (first != '-'))
-                                       ThisOpersModes = "+" + ThisOpersModes;
-                               if (ThisOpersModes != "")
+                               if (!ThisOpersModes.empty())
                                {
+                                       char first = *(ThisOpersModes.c_str());
+                                       if ((first != '+') && (first != '-'))
+                                               ThisOpersModes = "+" + ThisOpersModes;
+
                                        std::string buf;
                                        stringstream ss(ThisOpersModes);
-
                                        vector<string> tokens;
 
                                        // split ThisOperModes into modes and mode params
@@ -85,7 +85,7 @@ class ModuleModesOnOper : public Module
 
                                        // process mode params
                                        int i = 1;
-                                       for (unsigned int k = 1; k < tokens.size(); k++)
+                                       for (unsigned int k = 0; k < tokens.size(); k++)
                                        {
                                                modes[i] = tokens[k].c_str();
                                                i++;
@@ -106,8 +106,6 @@ class ModuleModesOnOper : public Module
        }
 };
 
-// stuff down here is the module-factory stuff. For basic modules you can ignore this.
-
 class ModuleModesOnOperFactory : public ModuleFactory
 {
  public: