Remove code for clientcount/servercount from /stats p. At the moment it was slow, hacky and doesn't totally work properly (bind 2 same ports on diff. ips and watch it get confused ;p). It may be added back in a few versions once we have the infrastructure in place to tie a list of clients to a port.

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

1 files changed, 6 insertions, 16 deletions

diff --git a/src/modules/m_spanningtree/main.cpp b/src/modules/m_spanningtree/main.cpp
index bebb69012..9e188b49d 100644
--- a/src/modules/m_spanningtree/main.cpp
+++ b/src/modules/m_spanningtree/main.cpp
@@ -661,22 +661,12 @@ int ModuleSpanningTree::OnStats(char statschar, userrec* user, string_list &resu
 
 		for (unsigned int i = 0; i < Utils->Bindings.size(); i++)
 		{
-			/*
-			 * XXX - todo:
-			 *  we need to remove duplicate ports from this list, and possibly also
-			 *  show the correct number of servers on the port.
-			 *
-			 * XXX also, this doesn't currently work, as something is not initialising ->port, so it's 0 all the time.
-			 *
-			 */
-			if (Utils->Bindings[i]->port)
-			{
-				std::string ip = Utils->Bindings[i]->IP;
-				if (ip.empty())
-					ip = "*";
-				results.push_back(ConvToStr(ServerInstance->Config->ServerName) + " 249 "+user->nick+" :" + ip + ":" + ConvToStr(Utils->Bindings[i]->port)+" (X servers) "+
-						ServerInstance->Config->ports[i]->GetDescription());
-			}
+			std::string ip = Utils->Bindings[i]->IP;
+			if (ip.empty())
+				ip = "*";
+
+			results.push_back(ConvToStr(ServerInstance->Config->ServerName) + " 249 "+user->nick+" :" + ip + ":" + ConvToStr(Utils->Bindings[i]->port)+
+				" (server, " + ServerInstance->Config->ports[i]->GetDescription() + ")");
 		}
 	}
 	return 0;