From d8f26f22609015ffde96cd57244e91c9998c0a7c Mon Sep 17 00:00:00 2001 From: smoogipoo Date: Thu, 29 Mar 2018 22:22:42 +0900 Subject: [PATCH] Make HitObjectMaskLayer not iterate through all masks when deselecting --- .../Edit/Screens/Compose/Layers/HitObjectMaskLayer.cs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/osu.Game/Screens/Edit/Screens/Compose/Layers/HitObjectMaskLayer.cs b/osu.Game/Screens/Edit/Screens/Compose/Layers/HitObjectMaskLayer.cs index 7b140ac37b..2907f48568 100644 --- a/osu.Game/Screens/Edit/Screens/Compose/Layers/HitObjectMaskLayer.cs +++ b/osu.Game/Screens/Edit/Screens/Compose/Layers/HitObjectMaskLayer.cs @@ -21,7 +21,7 @@ namespace osu.Game.Screens.Edit.Screens.Compose.Layers private readonly SelectionBox selectionBox; - private readonly HashSet selectedObjects = new HashSet(); + private readonly HashSet selectedMasks = new HashSet(); public HitObjectMaskLayer(Playfield playfield, HitObjectComposer composer) { @@ -88,9 +88,9 @@ namespace osu.Game.Screens.Edit.Screens.Compose.Layers selectionBox.RemoveMask(mask); } - private void onSelected(HitObjectMask mask) => selectedObjects.Add(mask); + private void onSelected(HitObjectMask mask) => selectedMasks.Add(mask); - private void onDeselected(HitObjectMask mask) => selectedObjects.Remove(mask); + private void onDeselected(HitObjectMask mask) => selectedMasks.Remove(mask); private void onSingleSelectionRequested(HitObjectMask mask) => DeselectAll(); @@ -103,6 +103,6 @@ namespace osu.Game.Screens.Edit.Screens.Compose.Layers /// /// Deselects all selected s. /// - public void DeselectAll() => maskContainer.ToList().ForEach(m => m.Deselect()); + public void DeselectAll() => selectedMasks.ToList().ForEach(m => m.Deselect()); } }