ClientList: use "delete" instead of Client::Close()

Client::Close() installs a TimeoutMonitor, which is not something we should do during shutdown.

ClientList: use "delete" instead of Client::Close()
4c705334 · Max Kellermann · c12da599 · 4c705334
Commit 4c705334 authored Jan 05, 2014 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 2 deletions

ClientList.cxx src/ClientList.cxx +8 -2

No files found.
--- a/src/ClientList.cxx
+++ b/src/ClientList.cxx
@@ -40,8 +40,14 @@ ClientList::Remove(Client &client)
 void
 ClientList::CloseAll()
 {
-	while (!list.empty())
-		list.front()->Close();
+	while (!list.empty()) {
+		delete list.front();
+		list.pop_front();
+
+#ifndef NDEBUG
+		--size;
+#endif
+	}

 	assert(size == 0);
 }