Minor ISupportManager changes

- Make GetLines() a const method
- Rename Lines to cachedlines
- Get rid of a variable in Build()

1 files changed, 3 insertions, 4 deletions

diff --git a/src/server.cpp b/src/server.cpp
index 66466fae9..ceefa4387 100644
--- a/src/server.cpp
+++ b/src/server.cpp
@@ -198,10 +198,9 @@ void ISupportManager::Build()
 	}
 
 	// 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();
+	cachedlines.clear();
 
 	for (std::map<std::string, std::string>::const_iterator it = tokens.begin(); it != tokens.end(); ++it)
 	{
@@ -220,7 +219,7 @@ void ISupportManager::Build()
 			// 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);
+			cachedlines.push_back(line);
 			line.clear();
 		}
 	}
@@ -228,6 +227,6 @@ void ISupportManager::Build()
 
 void ISupportManager::SendTo(LocalUser* user)
 {
-	for (std::vector<std::string>::const_iterator i = this->Lines.begin(); i != this->Lines.end(); ++i)
+	for (std::vector<std::string>::const_iterator i = cachedlines.begin(); i != cachedlines.end(); ++i)
 		user->WriteNumeric(RPL_ISUPPORT, *i);
 }