(svn r14719) -Codechange: replace DEREF_CLIENT with an instance function and replace looping socket structs with info structs when the loop is only interested in the info structs (i.e. not derefing the info from sockets when one can loop info directly and the socket isn't used)

1 files changed, 3 insertions, 3 deletions

diff --git a/src/network/network_gui.cpp b/src/network/network_gui.cpp
index f9d972c6b..d3f7a4ca5 100644
--- a/src/network/network_gui.cpp
+++ b/src/network/network_gui.cpp
@@ -1419,7 +1419,7 @@ static const NetworkClientInfo *NetworkFindClientInfo(byte client_no)
 {
 	const NetworkClientInfo *ci;
 
-	FOR_ALL_ACTIVE_CLIENT_INFOS(ci) {
+	FOR_ALL_CLIENT_INFOS(ci) {
 		if (client_no == 0) return ci;
 		client_no--;
 	}
@@ -1659,7 +1659,7 @@ struct NetworkClientListWindow : Window
 		const NetworkClientInfo *ci;
 
 		/* Should be replaced with a loop through all clients */
-		FOR_ALL_ACTIVE_CLIENT_INFOS(ci) {
+		FOR_ALL_CLIENT_INFOS(ci) {
 			num++;
 		}
 
@@ -1689,7 +1689,7 @@ struct NetworkClientListWindow : Window
 
 		int y = CLNWND_OFFSET;
 
-		FOR_ALL_ACTIVE_CLIENT_INFOS(ci) {
+		FOR_ALL_CLIENT_INFOS(ci) {
 			TextColour colour;
 			if (this->selected_item == i++) { // Selected item, highlight it
 				GfxFillRect(1, y, 248, y + CLNWND_ROWSIZE - 1, 0);