X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fserver.cpp;h=efefa0d37a4b713951057708e8ffc7844db63124;hb=68afa315fccf5d741103137cb65d613a4e9bfef1;hp=c2068d75bb169e4e084f30734c5848137afba6e7;hpb=e586aaab7c4f7b03514c83451d73b73f55dc6998;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/server.cpp b/src/server.cpp
index c2068d75b..efefa0d37 100644
--- a/src/server.cpp
+++ b/src/server.cpp
@@ -36,7 +36,7 @@ void InspIRCd::SignalHandler(int signal)
 	else if (signal == SIGTERM)
 #endif
 	{
-		Exit(signal);
+		Exit(EXIT_STATUS_SIGTERM);
 	}
 }
 
@@ -58,7 +58,7 @@ void InspIRCd::Exit(int status)
 void RehashHandler::Call(const std::string &reason)
 {
 	ServerInstance->SNO->WriteToSnoMask('a', "Rehashing config file %s %s",ServerConfig::CleanFilename(ServerInstance->ConfigFileName.c_str()), reason.c_str());
-	FOREACH_MOD(I_OnGarbageCollect, OnGarbageCollect());
+	FOREACH_MOD(OnGarbageCollect, ());
 	if (!ServerInstance->ConfigThread)
 	{
 		ServerInstance->ConfigThread = new ConfigReaderThread("");
@@ -73,11 +73,6 @@ std::string InspIRCd::GetVersionString(bool getFullVersion)
 	return BRANCH " " + Config->ServerName + " :" + Config->CustomVersion;
 }
 
-const char InspIRCd::LogHeader[] =
-	"Log started for " VERSION " (" REVISION ", " MODULE_INIT_STR ")"
-	" - compiled on " SYSTEM;
-
-
 std::string UIDGenerator::GenerateSID(const std::string& servername, const std::string& serverdesc)
 {
 	unsigned int sid = 0;
@@ -88,6 +83,7 @@ std::string UIDGenerator::GenerateSID(const std::string& servername, const std::
 		sid = 5 * sid + *i;
 
 	std::string sidstr = ConvToStr(sid % 1000);
+	sidstr.insert(0, 3 - sidstr.length(), '0');
 	return sidstr;
 }
 
@@ -171,13 +167,10 @@ void ISupportManager::Build()
 	 * the future we can use an initializer list here.
 	 */
 	std::map<std::string, std::string> tokens;
-	std::vector<std::string> lines;
-	int token_count = 0;
-	std::string line;
 
 	tokens["AWAYLEN"] = ConvToStr(ServerInstance->Config->Limits.MaxAway);
 	tokens["CASEMAPPING"] = "rfc1459";
-	tokens["CHANMODES"] = ConvToStr(ServerInstance->Modes->GiveModeList(MASK_CHANNEL));
+	tokens["CHANMODES"] = ServerInstance->Modes->GiveModeList(MASK_CHANNEL);
 	tokens["CHANNELLEN"] = ConvToStr(ServerInstance->Config->Limits.ChanMax);
 	tokens["CHANTYPES"] = "#";
 	tokens["CHARSET"] = "ascii";
@@ -185,10 +178,9 @@ void ISupportManager::Build()
 	tokens["KICKLEN"] = ConvToStr(ServerInstance->Config->Limits.MaxKick);
 	tokens["MAXBANS"] = "64"; // TODO: make this a config setting.
 	tokens["MAXCHANNELS"] = ConvToStr(ServerInstance->Config->MaxChans);
-	tokens["MAXPARA"] = ConvToStr(MAXPARAMETERS);
 	tokens["MAXTARGETS"] = ConvToStr(ServerInstance->Config->MaxTargets);
 	tokens["MODES"] = ConvToStr(ServerInstance->Config->Limits.MaxModes);
-	tokens["NETWORK"] = ConvToStr(ServerInstance->Config->Network);
+	tokens["NETWORK"] = ServerInstance->Config->Network;
 	tokens["NICKLEN"] = ConvToStr(ServerInstance->Config->Limits.NickMax);
 	tokens["PREFIX"] = ServerInstance->Modes->BuildPrefixes();
 	tokens["STATUSMSG"] = ServerInstance->Modes->BuildPrefixes(false);
@@ -197,7 +189,8 @@ void ISupportManager::Build()
 	tokens["FNC"] = tokens["MAP"] = tokens["VBANLIST"] =
 		tokens["WALLCHOPS"] = tokens["WALLVOICES"];
 
-	FOREACH_MOD(I_On005Numeric, On005Numeric(tokens));
+	// Modules can add new tokens and also edit or remove existing tokens
+	FOREACH_MOD(On005Numeric, (tokens));
 
 	// EXTBAN is a special case as we need to sort it and prepend a comma.
 	std::map<std::string, std::string>::iterator extban = tokens.find("EXTBAN");
@@ -207,24 +200,37 @@ void ISupportManager::Build()
 		extban->second.insert(0, ",");
 	}
 
-	for (std::map<std::string, std::string>::iterator it = tokens.begin(); it != tokens.end(); it++)
+	// Transform the map into a list of lines, ready to be sent to clients
+	std::vector<std::string>& lines = this->Lines;
+	std::string line;
+	unsigned int token_count = 0;
+	lines.clear();
+
+	for (std::map<std::string, std::string>::const_iterator it = tokens.begin(); it != tokens.end(); ++it)
 	{
-		line.append(it->first + (it->second.empty() ? " " : "=" + it->second + " "));
+		line.append(it->first);
+
+		// If this token has a value then append a '=' char after the name and then the value itself
+		if (!it->second.empty())
+			line.append(1, '=').append(it->second);
+
+		// Always append a space, even if it's the last token because all lines will be suffixed
+		line.push_back(' ');
 		token_count++;
 
 		if (token_count % 13 == 12 || it == --tokens.end())
 		{
+			// Reached maximum number of tokens for this line or the current token
+			// is the last one; finalize the line and store it for later use
 			line.append(":are supported by this server");
 			lines.push_back(line);
 			line.clear();
 		}
 	}
-
-	this->Lines = lines;
 }
 
 void ISupportManager::SendTo(LocalUser* user)
 {
-	for (std::vector<std::string>::iterator line = this->Lines.begin(); line != this->Lines.end(); line++)
-		user->WriteNumeric(RPL_ISUPPORT, "%s %s", user->nick.c_str(), line->c_str());
+	for (std::vector<std::string>::const_iterator i = this->Lines.begin(); i != this->Lines.end(); ++i)
+		user->WriteNumeric(RPL_ISUPPORT, "%s %s", user->nick.c_str(), i->c_str());
 }