Remove unnecessary UnbindAdjustments call

It is not necessary given that `CurrentTrack` already removes all adjustments first.
Lawtrohux · May 10, 2022 · 9446be2 · 9446be2
1 parent 36a7644
commit 9446be2
Showing 1 changed file with 2 additions and 12 deletions.
diff --git a/osu.Game/Overlays/MusicController.cs b/osu.Game/Overlays/MusicController.cs
@@ -393,21 +393,11 @@ public void ResetTrackAdjustments()
             CurrentTrack.RemoveAllAdjustments(AdjustableProperty.Tempo);
             CurrentTrack.RemoveAllAdjustments(AdjustableProperty.Volume);
 
-            var applicableToTrack = mods.Value.OfType<IApplicableToTrack>();
-
-            if (!allowTrackAdjustments || !applicableToTrack.Any())
-            {
-                if (modTrackAdjustments != null)
-                {
-                    CurrentTrack.UnbindAdjustments(modTrackAdjustments);
-                    modTrackAdjustments = null;
-                }
-            }
-            else
+            if (allowTrackAdjustments)
             {
                 CurrentTrack.BindAdjustments(modTrackAdjustments = new AudioAdjustments());
 
-                foreach (var mod in applicableToTrack)
+                foreach (var mod in mods.Value.OfType<IApplicableToTrack>())
                     mod.ApplyToTrack(modTrackAdjustments);
             }
         }