h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2025-01-15 09:22:54 +08:00
Code Issues Releases Wiki Activity

Don't trigger the same exception multiple times

Browse Source
This commit is contained in:
smoogipoo 2020-06-27 00:19:22 +09:00
parent c233dc4768
commit e8d36bc3cb
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
osu.Game/Screens/Multi/RoomManager.cs
View File

@ -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);
}
}