Faster CullItem/CullList using strdup rather than copying a std::string

git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@3626 e03df62e-2008-0410-955e-edbf42e46eb7

1 files changed, 9 insertions, 4 deletions

diff --git a/src/cull_list.cpp b/src/cull_list.cpp
index b7254fd0b..0951a3224 100644
--- a/src/cull_list.cpp
+++ b/src/cull_list.cpp
@@ -84,13 +84,18 @@ bool CullList::IsValid(userrec* user)
 CullItem::CullItem(userrec* u, std::string &r)
 {
         this->user = u;
-        this->reason = r;
+        this->reason = strdup(r.c_str());
 }
 
 CullItem::CullItem(userrec* u, const char* r)
 {
 	this->user = u;
-	this->reason = r;
+	this->reason = strdup(r);
+}
+
+CullItem::~CullItem()
+{
+	free(reason);
 }
 
 userrec* CullItem::GetUser()
@@ -98,7 +103,7 @@ userrec* CullItem::GetUser()
         return this->user;
 }
 
-std::string CullItem::GetReason()
+const char* CullItem::GetReason()
 {
         return this->reason;
 }
@@ -145,7 +150,7 @@ int CullList::Apply()
 		 */
 		if (IsValid(u))
 		{
-			kill_link(u,a->GetReason().c_str());
+			kill_link(u,a->GetReason());
 			list.erase(list.begin());
 			/* So that huge numbers of quits dont block,
 			 * we yield back to our mainloop every 15