From 83358d487fb5770877948e671621eada6cc91abb Mon Sep 17 00:00:00 2001
From: Dean Herbert <pe@ppy.sh>
Date: Tue, 13 Oct 2020 18:18:22 +0900
Subject: [PATCH] Remove logging

---
 osu.Game/Screens/Select/BeatmapCarousel.cs    | 20 ++++++++-----------
 .../Carousel/DrawableCarouselBeatmapSet.cs    |  5 -----
 2 files changed, 8 insertions(+), 17 deletions(-)

diff --git a/osu.Game/Screens/Select/BeatmapCarousel.cs b/osu.Game/Screens/Select/BeatmapCarousel.cs
index 955693a67c..e53beaef0b 100644
--- a/osu.Game/Screens/Select/BeatmapCarousel.cs
+++ b/osu.Game/Screens/Select/BeatmapCarousel.cs
@@ -1,30 +1,29 @@
 // Copyright (c) ppy Pty Ltd <contact@ppy.sh>. Licensed under the MIT Licence.
 // See the LICENCE file in the repository root for full licence text.
 
-using osuTK;
-using osu.Framework.Graphics;
-using osu.Framework.Graphics.Containers;
 using System;
 using System.Collections.Generic;
-using System.Linq;
-using osu.Game.Configuration;
-using osuTK.Input;
-using osu.Framework.Utils;
 using System.Diagnostics;
+using System.Linq;
 using osu.Framework.Allocation;
 using osu.Framework.Bindables;
 using osu.Framework.Caching;
-using osu.Framework.Threading;
+using osu.Framework.Graphics;
+using osu.Framework.Graphics.Containers;
 using osu.Framework.Graphics.Pooling;
 using osu.Framework.Input.Bindings;
 using osu.Framework.Input.Events;
-using osu.Framework.Logging;
+using osu.Framework.Threading;
+using osu.Framework.Utils;
 using osu.Game.Beatmaps;
+using osu.Game.Configuration;
 using osu.Game.Extensions;
 using osu.Game.Graphics.Containers;
 using osu.Game.Graphics.Cursor;
 using osu.Game.Input.Bindings;
 using osu.Game.Screens.Select.Carousel;
+using osuTK;
+using osuTK.Input;
 
 namespace osu.Game.Screens.Select
 {
@@ -589,8 +588,6 @@ namespace osu.Game.Screens.Select
             // This involves fetching new items from the pool, returning no-longer required items.
             if (revalidateItems || newDisplayRange != displayedRange)
             {
-                Logger.Log("revalidation requested");
-
                 displayedRange = newDisplayRange;
 
                 var toDisplay = visibleItems.GetRange(displayedRange.first, displayedRange.last - displayedRange.first);
@@ -616,7 +613,6 @@ namespace osu.Game.Screens.Select
                 // Add those items within the previously found index range that should be displayed.
                 foreach (var item in toDisplay)
                 {
-                    Logger.Log($"getting panel for {item} from pool");
                     var panel = setPool.Get(p => p.Item = item);
 
                     panel.Depth = item.CarouselYPosition;
diff --git a/osu.Game/Screens/Select/Carousel/DrawableCarouselBeatmapSet.cs b/osu.Game/Screens/Select/Carousel/DrawableCarouselBeatmapSet.cs
index b80e789429..bf76adc6ac 100644
--- a/osu.Game/Screens/Select/Carousel/DrawableCarouselBeatmapSet.cs
+++ b/osu.Game/Screens/Select/Carousel/DrawableCarouselBeatmapSet.cs
@@ -9,7 +9,6 @@ using osu.Framework.Graphics;
 using osu.Framework.Graphics.Containers;
 using osu.Framework.Graphics.Cursor;
 using osu.Framework.Graphics.UserInterface;
-using osu.Framework.Logging;
 using osu.Game.Beatmaps;
 using osu.Game.Collections;
 using osu.Game.Graphics.UserInterface;
@@ -70,12 +69,10 @@ namespace osu.Game.Screens.Select.Carousel
             if (Item == null)
                 return;
 
-            Logger.Log($"updating item {beatmapSet}");
             Header.Children = new Drawable[]
             {
                 new DelayedLoadUnloadWrapper(() =>
                 {
-                    Logger.Log($"loaded background item {beatmapSet}");
                     var background = new SetPanelBackground(manager.GetWorkingBeatmap(beatmapSet.Beatmaps.FirstOrDefault()))
                     {
                         RelativeSizeAxes = Axes.Both,
@@ -151,8 +148,6 @@ namespace osu.Game.Screens.Select.Carousel
                     ChildrenEnumerable = visibleBeatmaps.Select(c => c.CreateDrawableRepresentation()).ToArray()
                 };
 
-                Logger.Log($"loading {visibleBeatmaps.Length} beatmaps for {Item}");
-
                 LoadComponentAsync(beatmapContainer, loaded =>
                 {
                     // make sure the pooled target hasn't changed.