m_namesx, m_uhnames Don't convert the command name to irc::string in OnPreCommand()

The command is converted into an uppercase string in CommandParser::ProcessCommand()

diff --git a/src/modules/m_namesx.cpp b/src/modules/m_namesx.cpp
index 7e9c5fd48f6e6dbaaf586d74559308dbe91c4d75..535f6ec103e598e77feb44a7a053fde194a744eb 100644 (file)
--- a/src/modules/m_namesx.cpp
+++ b/src/modules/m_namesx.cpp
@@ -52,13 +52,12 @@ class ModuleNamesX : public Module
 
        ModResult OnPreCommand(std::string &command, std::vector<std::string> &parameters, LocalUser *user, bool validated, const std::string &original_line)
        {
-               irc::string c = command.c_str();
                /* We don't actually create a proper command handler class for PROTOCTL,
                 * because other modules might want to have PROTOCTL hooks too.
                 * Therefore, we just hook its as an unvalidated command therefore we
                 * can capture it even if it doesnt exist! :-)
                 */
-               if (c == "PROTOCTL")
+               if (command == "PROTOCTL")
                {
                        if ((parameters.size()) && (!strcasecmp(parameters[0].c_str(),"NAMESX")))
                        {

diff --git a/src/modules/m_uhnames.cpp b/src/modules/m_uhnames.cpp
index 37bd674408bf6072c46c14de7b63a6441e815689..5693eb8070b8006a9907f55bfbf4af0cf256e808 100644 (file)
--- a/src/modules/m_uhnames.cpp
+++ b/src/modules/m_uhnames.cpp
@@ -51,13 +51,12 @@ class ModuleUHNames : public Module
 
        ModResult OnPreCommand(std::string &command, std::vector<std::string> &parameters, LocalUser *user, bool validated, const std::string &original_line)
        {
-               irc::string c = command.c_str();
                /* We don't actually create a proper command handler class for PROTOCTL,
                 * because other modules might want to have PROTOCTL hooks too.
                 * Therefore, we just hook its as an unvalidated command therefore we
                 * can capture it even if it doesnt exist! :-)
                 */
-               if (c == "PROTOCTL")
+               if (command == "PROTOCTL")
                {
                        if ((parameters.size()) && (!strcasecmp(parameters[0].c_str(),"UHNAMES")))
                        {