*TEST CODE* Faster custom mode set/unset

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

2 files changed, 30 insertions, 76 deletions

diff --git a/src/channels.cpp b/src/channels.cpp
index f086b9853..3b05cf8ea 100644
--- a/src/channels.cpp
+++ b/src/channels.cpp
@@ -66,104 +66,59 @@ extern chan_hash chanlist;
 
 using namespace std;
 
-//std::vector<ModeParameter> custom_mode_params;
-
 chanrec* ForceChan(chanrec* Ptr,ucrec &a,userrec* user, int created);
 
 chanrec::chanrec()
 {
-	*name = *custom_modes = *topic = *setby = *key = 0;
+	*name = *topic = *setby = *key = 0;
 	created = topicset = limit = binarymodes = 0;
 	internal_userlist.clear();
+	memset(&custom_modes,0,190);
 }
 
 void chanrec::SetCustomMode(char mode,bool mode_on)
 {
-	if (mode_on)
-	{
-		char* mptr = this->custom_modes;
-		int ssize = 0;
-
-		/* Attempt to find the end of the mode string */
-		while (*mptr++)
-		{
-			/* While iterating the mode string, we found that they already have
-			 * this mode in their list. Abort right now. */
-			if (*mptr == mode)
-				return;
-			/* Increment the string size, saves us doing strlen */
-			ssize++;
-		}
-
-		log(DEBUG,"ssize=%d",ssize);
-
-		/* Is there room left in the buffer? If there is append the mode */
-		if (ssize < MAXMODES-1)
-		{
-			*--mptr = mode;
-			*++mptr = 0;
-		}
-
-		log(DEBUG,"Custom mode %c set, modes='%s'",mode,this->custom_modes);
-	}
-	else
-	{
-		if (charremove(this->custom_modes,mode))
-		{
-			log(DEBUG,"Custom mode %c removed: modelist='%s'",mode,this->custom_modes);
-			/* Only call this if we found the mode */
-			this->SetCustomModeParam(mode,"",false);
-		}
-	}
+	custom_modes[mode-65] = mode_on;
+	if (!mode_on)
+		this->SetCustomModeParam(mode,"",false);
 }
 
 
 void chanrec::SetCustomModeParam(char mode,char* parameter,bool mode_on)
 {
-
 	log(DEBUG,"SetCustomModeParam called");
-	ModeParameter M;
-	M.mode = mode;
-	strlcpy(M.channel,this->name,CHANMAX);
-	strlcpy(M.parameter,parameter,MAXBUF);
+	
+	std::map<char,char*>::iterator n = custom_mode_params.find(mode);	
+
 	if (mode_on)
 	{
 		log(DEBUG,"Custom mode parameter %c %s added",mode,parameter);
-		custom_mode_params.push_back(M);
+		if (n == custom_mode_params.end())
+		{
+			custom_mode_params[mode] = strdup(parameter);
+		}
 	}
 	else
 	{
-		if (custom_mode_params.size())
+		if (n != custom_mode_params.end())
 		{
-			for (vector<ModeParameter>::iterator i = custom_mode_params.begin(); i < custom_mode_params.end(); i++)
-			{
-				if ((i->mode == mode) && (!strcasecmp(this->name,i->channel)))
-				{
-					log(DEBUG,"Custom mode parameter %c %s removed",mode,parameter);
-					custom_mode_params.erase(i);
-					return;
-				}
-			}
+			free(n->second);
+			custom_mode_params.erase(n);
 		}
 	}
 }
 
 bool chanrec::IsCustomModeSet(char mode)
 {
-	return (strchr(this->custom_modes,mode));
+	return custom_modes[mode-65];
 }
 
 std::string chanrec::GetModeParameter(char mode)
 {
-	if (custom_mode_params.size())
+	std::map<char,char*>::iterator n = custom_mode_params.find(mode);
+	if (n != custom_mode_params.end())
 	{
-		for (vector<ModeParameter>::iterator i = custom_mode_params.begin(); i < custom_mode_params.end(); i++)
-		{
-			if ((i->mode == mode) && (!strcasecmp(this->name,i->channel)))
-			{
-				return i->parameter;
-			}
-		}
+		return n->second;
 	}
 	return "";
 }
diff --git a/src/helperfuncs.cpp b/src/helperfuncs.cpp
index 854675378..8562be443 100644
--- a/src/helperfuncs.cpp
+++ b/src/helperfuncs.cpp
@@ -1198,19 +1198,18 @@ char* chanmodes(chanrec *chan, bool showkey)
                 sprintf(foo," %lu",(unsigned long)chan->limit);
                 strlcat(sparam,foo,MAXBUF);
         }
-        if (*chan->custom_modes)
+        for (int n = 0; n < 190; n++)
         {
-		for (char* t = chan->custom_modes; *t; t++)
-	                *offset++ = *t;
-                for (int z = 0; chan->custom_modes[z]; z++)
-                {
-                        std::string extparam = chan->GetModeParameter(chan->custom_modes[z]);
-                        if (extparam != "")
-                        {
-                                charlcat(sparam,' ',MAXBUF);
-                                strlcat(sparam,extparam.c_str(),MAXBUF);
-                        }
-                }
+		if (custom_modes[n])
+		{
+	                *offset++ = n+65;
+			std::string extparam = chan->GetModeParameter(n+65);
+			if (extparam != "")
+			{
+				charlcat(sparam,' ',MAXBUF);
+				strlcat(sparam,extparam.c_str(),MAXBUF);
+			}
+		}
         }
 	/* Null terminate scratch */
 	*offset = '\0';