From e8d36bc3cbb5c6aa2c0f0fd6dde7d18d34799590 Mon Sep 17 00:00:00 2001
From: smoogipoo <smoogipoo@smgi.me>
Date: Sat, 27 Jun 2020 00:19:22 +0900
Subject: [PATCH] Don't trigger the same exception multiple times

---
 osu.Game/Screens/Multi/RoomManager.cs | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/osu.Game/Screens/Multi/RoomManager.cs b/osu.Game/Screens/Multi/RoomManager.cs
index b8c969a845..5083fb2ee3 100644
--- a/osu.Game/Screens/Multi/RoomManager.cs
+++ b/osu.Game/Screens/Multi/RoomManager.cs
@@ -3,6 +3,7 @@
 
 using System;
 using System.Collections.Generic;
+using System.Diagnostics;
 using System.Linq;
 using System.Threading.Tasks;
 using osu.Framework.Allocation;
@@ -142,6 +143,8 @@ namespace osu.Game.Screens.Multi
             joinedRoom = null;
         }
 
+        private readonly List<int> roomsFailedUpdate = new List<int>();
+
         /// <summary>
         /// Invoked when the listing of all <see cref="Room"/>s is received from the server.
         /// </summary>
@@ -173,7 +176,14 @@ namespace osu.Game.Screens.Multi
                 }
                 catch (Exception ex)
                 {
-                    Logger.Error(ex, $"Failed to update room: {r.Name.Value}.");
+                    Debug.Assert(r.RoomID.Value != null);
+
+                    if (!roomsFailedUpdate.Contains(r.RoomID.Value.Value))
+                    {
+                        Logger.Error(ex, $"Failed to update room: {r.Name.Value}.");
+                        roomsFailedUpdate.Add(r.RoomID.Value.Value);
+                    }
+
                     rooms.Remove(r);
                 }
             }