More numerics.h conversion

[user/henk/code/inspircd.git] / src / commands / cmd_topic.cpp

diff --git a/src/commands/cmd_topic.cpp b/src/commands/cmd_topic.cpp
index 9f9d56b147572a2da06abd8e44be6f427753d470..0e1a115da81494a0f7f035dd6574414c16515808 100644 (file)
--- a/src/commands/cmd_topic.cpp
+++ b/src/commands/cmd_topic.cpp
@@ -31,22 +31,22 @@ CmdResult CommandTopic::Handle (const std::vector<std::string>& parameters, User
                {
                        if ((Ptr->IsModeSet('s')) && (!Ptr->HasUser(user)))
                        {
-                               user->WriteNumeric(401, "%s %s :No such nick/channel",user->nick, Ptr->name);
+                               user->WriteNumeric(401, "%s %s :No such nick/channel",user->nick.c_str(), Ptr->name.c_str());
                                return CMD_FAILURE;
                        }
                        if (Ptr->topicset)
                        {
-                               user->WriteNumeric(332, "%s %s :%s", user->nick, Ptr->name, Ptr->topic);
-                               user->WriteNumeric(333, "%s %s %s %lu", user->nick, Ptr->name, Ptr->setby, (unsigned long)Ptr->topicset);
+                               user->WriteNumeric(332, "%s %s :%s", user->nick.c_str(), Ptr->name.c_str(), Ptr->topic.c_str());
+                               user->WriteNumeric(333, "%s %s %s %lu", user->nick.c_str(), Ptr->name.c_str(), Ptr->setby.c_str(), (unsigned long)Ptr->topicset);
                        }
                        else
                        {
-                               user->WriteNumeric(331, "%s %s :No topic is set.", user->nick, Ptr->name);
+                               user->WriteNumeric(331, "%s %s :No topic is set.", user->nick.c_str(), Ptr->name.c_str());
                        }
                }
                else
                {
-                       user->WriteNumeric(401, "%s %s :No such nick/channel",user->nick, parameters[0].c_str());
+                       user->WriteNumeric(401, "%s %s :No such nick/channel",user->nick.c_str(), parameters[0].c_str());
                        return CMD_FAILURE;
                }
                return CMD_SUCCESS;
@@ -60,17 +60,17 @@ CmdResult CommandTopic::Handle (const std::vector<std::string>& parameters, User
                        {
                                if (!Ptr->HasUser(user))
                                {
-                                       user->WriteNumeric(442, "%s %s :You're not on that channel!",user->nick, Ptr->name);
+                                       user->WriteNumeric(442, "%s %s :You're not on that channel!",user->nick.c_str(), Ptr->name.c_str());
                                        return CMD_FAILURE;
                                }
                                if ((Ptr->IsModeSet('t')) && (Ptr->GetStatus(user) < STATUS_HOP))
                                {
-                                       user->WriteNumeric(482, "%s %s :You must be at least a half-operator to change the topic on this channel", user->nick, Ptr->name);
+                                       user->WriteNumeric(482, "%s %s :You must be at least a half-operator to change the topic on this channel", user->nick.c_str(), Ptr->name.c_str());
                                        return CMD_FAILURE;
                                }
                        }
 
-                       char topic[MAXTOPIC];
+                       std::string topic;
 
                        if (IS_LOCAL(user))
                        {
@@ -79,26 +79,25 @@ CmdResult CommandTopic::Handle (const std::vector<std::string>& parameters, User
                                 */
                                int MOD_RESULT = 0;
 
-                               strlcpy(topic, parameters[1].c_str(), MAXTOPIC-1);
+                               topic.assign(parameters[1], 0, ServerInstance->Config->Limits.MaxTopic);
                                FOREACH_RESULT(I_OnLocalTopicChange,OnLocalTopicChange(user,Ptr,topic));
                                if (MOD_RESULT)
                                        return CMD_FAILURE;
 
-                               strlcpy(Ptr->topic, topic, MAXTOPIC-1);
+                               Ptr->topic.assign(topic, 0, ServerInstance->Config->Limits.MaxTopic);
                        }
                        else
                        {
                                /* Sneaky shortcut, one string copy for a remote topic */
-                               strlcpy(Ptr->topic, parameters[1].c_str(), MAXTOPIC-1);
+                               Ptr->topic.assign(parameters[1], 0, ServerInstance->Config->Limits.MaxTopic);
                        }
 
-                       if (ServerInstance->Config->FullHostInTopic)
-                               strlcpy(Ptr->setby,user->GetFullHost(),127);
-                       else
-                               strlcpy(Ptr->setby,user->nick,127);
+                       Ptr->setby.assign(ServerInstance->Config->FullHostInTopic ?
+                                       user->GetFullHost() : user->nick,
+                                       0, 128);
 
                        Ptr->topicset = ServerInstance->Time();
-                       Ptr->WriteChannel(user, "TOPIC %s :%s", Ptr->name, Ptr->topic);
+                       Ptr->WriteChannel(user, "TOPIC %s :%s", Ptr->name.c_str(), Ptr->topic.c_str());
 
                        if (IS_LOCAL(user))
                                /* We know 'topic' will contain valid data here */
@@ -106,7 +105,7 @@ CmdResult CommandTopic::Handle (const std::vector<std::string>& parameters, User
                }
                else
                {
-                       user->WriteNumeric(401, "%s %s :No such nick/channel",user->nick, parameters[0].c_str());
+                       user->WriteNumeric(401, "%s %s :No such nick/channel",user->nick.c_str(), parameters[0].c_str());
                        return CMD_FAILURE;
                }
        }