From ddadf3c4b5ecc2306cf99b9882ef5ba92dcb88b9 Mon Sep 17 00:00:00 2001 From: Dean Herbert Date: Tue, 21 Dec 2021 14:37:23 +0900 Subject: [PATCH 1/2] Add logging of `MultiplayerMatchSubScreen` exiting from room loss --- .../OnlinePlay/Multiplayer/MultiplayerMatchSubScreen.cs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/osu.Game/Screens/OnlinePlay/Multiplayer/MultiplayerMatchSubScreen.cs b/osu.Game/Screens/OnlinePlay/Multiplayer/MultiplayerMatchSubScreen.cs index 6895608c8e..ad332dd33a 100644 --- a/osu.Game/Screens/OnlinePlay/Multiplayer/MultiplayerMatchSubScreen.cs +++ b/osu.Game/Screens/OnlinePlay/Multiplayer/MultiplayerMatchSubScreen.cs @@ -11,6 +11,7 @@ using osu.Framework.Allocation; using osu.Framework.Bindables; using osu.Framework.Graphics; using osu.Framework.Graphics.Containers; +using osu.Framework.Logging; using osu.Framework.Screens; using osu.Framework.Threading; using osu.Game.Beatmaps; @@ -434,6 +435,8 @@ namespace osu.Game.Screens.OnlinePlay.Multiplayer private void handleRoomLost() => Schedule(() => { + Logger.Log($"{nameof(MultiplayerMatchSubScreen)} exiting due to loss of room or connection"); + if (this.IsCurrentScreen()) this.Exit(); else From d08b66a5ffa261731a001b540a903e37fc07c1b6 Mon Sep 17 00:00:00 2001 From: Dean Herbert Date: Tue, 21 Dec 2021 15:07:06 +0900 Subject: [PATCH 2/2] Also add logging to `OnlinePlayScreen` --- .../Screens/OnlinePlay/Multiplayer/MultiplayerMatchSubScreen.cs | 2 +- osu.Game/Screens/OnlinePlay/OnlinePlayScreen.cs | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/osu.Game/Screens/OnlinePlay/Multiplayer/MultiplayerMatchSubScreen.cs b/osu.Game/Screens/OnlinePlay/Multiplayer/MultiplayerMatchSubScreen.cs index ad332dd33a..af83543b16 100644 --- a/osu.Game/Screens/OnlinePlay/Multiplayer/MultiplayerMatchSubScreen.cs +++ b/osu.Game/Screens/OnlinePlay/Multiplayer/MultiplayerMatchSubScreen.cs @@ -435,7 +435,7 @@ namespace osu.Game.Screens.OnlinePlay.Multiplayer private void handleRoomLost() => Schedule(() => { - Logger.Log($"{nameof(MultiplayerMatchSubScreen)} exiting due to loss of room or connection"); + Logger.Log($"{this} exiting due to loss of room or connection"); if (this.IsCurrentScreen()) this.Exit(); diff --git a/osu.Game/Screens/OnlinePlay/OnlinePlayScreen.cs b/osu.Game/Screens/OnlinePlay/OnlinePlayScreen.cs index 19153521cd..bf1699dca0 100644 --- a/osu.Game/Screens/OnlinePlay/OnlinePlayScreen.cs +++ b/osu.Game/Screens/OnlinePlay/OnlinePlayScreen.cs @@ -95,6 +95,8 @@ namespace osu.Game.Screens.OnlinePlay private void forcefullyExit() { + Logger.Log($"{this} forcefully exiting due to loss of API connection"); + // This is temporary since we don't currently have a way to force screens to be exited if (this.IsCurrentScreen()) {