performance boost

Author: NeoCoderMatrix86

AudioCuesheetEditor.Tests/Services/IO/ImportManagerTests.cs

@@ -18,6 +18,7 @@
 using AudioCuesheetEditor.Model.AudioCuesheet.Import;
 using AudioCuesheetEditor.Model.IO;
 using AudioCuesheetEditor.Model.IO.Import;
+using AudioCuesheetEditor.Model.UI;
 using AudioCuesheetEditor.Services.IO;
 using AudioCuesheetEditor.Services.UI;
 using AudioCuesheetEditor.Tests.Utility;
@@ -248,8 +249,7 @@ public void ImportCuesheet_WithImportCuesheetAvailable_ImportsCuesheetData()
             Assert.AreEqual(analyzedCuesheet.Tracks.Last().Title, sessionStateContainer.Cuesheet.Tracks.Last().Title);
             Assert.AreEqual(analyzedCuesheet.Tracks.Last().Begin, sessionStateContainer.Cuesheet.Tracks.Last().Begin);
             Assert.AreEqual(analyzedCuesheet.Tracks.Last().End, sessionStateContainer.Cuesheet.Tracks.Last().End);
-            traceChangeManagerMock.Verify(x => x.RemoveTracedChanges(It.IsAny<Cuesheet>()));
-            traceChangeManagerMock.Verify(x => x.RemoveTracedChanges(It.IsAny<Track>()));
+            traceChangeManagerMock.Verify(x => x.RemoveTracedChanges(It.IsAny<IEnumerable<ITraceable>>()));
         }
 
         private static IBrowserFile CreateBrowserFileMock(string name, string content = "TestContent")

AudioCuesheetEditor.Tests/Services/UI/TraceChangeManagerTests.cs

@@ -374,7 +374,7 @@ public void RemoveTracedChanges_RemovesChanges_WhenChangesAvailable()
                 tracedObjectHistoryChangedFired = true;
             };
             // Act
-            manager.RemoveTracedChanges(cuesheet1);
+            manager.RemoveTracedChanges([cuesheet1]);
             // Assert
             Assert.IsTrue(tracedObjectHistoryChangedFired);
             Assert.IsTrue(manager.CanUndo);
@@ -400,7 +400,7 @@ public void RemoveTracedChanges_RemovesNoChanges_WhenNoChangesAvailable()
                 tracedObjectHistoryChangedFired = true;
             };
             // Act
-            manager.RemoveTracedChanges(cuesheet1);
+            manager.RemoveTracedChanges([cuesheet1]);
             // Assert
             Assert.IsTrue(tracedObjectHistoryChangedFired);
             Assert.IsTrue(manager.CanUndo);

AudioCuesheetEditor/Services/IO/ImportManager.cs

@@ -18,6 +18,7 @@
 using AudioCuesheetEditor.Model.IO;
 using AudioCuesheetEditor.Model.IO.Audio;
 using AudioCuesheetEditor.Model.IO.Import;
+using AudioCuesheetEditor.Model.UI;
 using AudioCuesheetEditor.Services.UI;
 using Microsoft.AspNetCore.Components.Forms;
 using System.Diagnostics;
@@ -237,11 +238,7 @@ private void ResetTracing()
         {
             if (_sessionStateContainer.ImportCuesheet != null)
             {
-                _traceChangeManager.RemoveTracedChanges(_sessionStateContainer.ImportCuesheet);
-                foreach (var track in _sessionStateContainer.ImportCuesheet.Tracks)
-                {
-                    _traceChangeManager.RemoveTracedChanges(track);
-                }
+                _traceChangeManager.RemoveTracedChanges([_sessionStateContainer.ImportCuesheet, .. _sessionStateContainer.ImportCuesheet.Tracks]);
             }
         }
     }

AudioCuesheetEditor/Services/UI/ITraceChangeManager.cs

@@ -24,6 +24,6 @@ public interface ITraceChangeManager
         public void Undo();
         public void Redo();
         public void MergeLastEditWithEdit(Func<TracedChanges, bool> targetEdit);
-        public void RemoveTracedChanges(ITraceable traceable);
+        public void RemoveTracedChanges(IEnumerable<ITraceable> traceables);
     }
 }

AudioCuesheetEditor/Services/UI/TraceChangeManager.cs

@@ -228,12 +228,12 @@ public void MergeLastEditWithEdit(Func<TracedChanges, bool> targetEdit)
             }
         }
 
-        public void RemoveTracedChanges(ITraceable traceable)
+        public void RemoveTracedChanges(IEnumerable<ITraceable> traceables)
         {
             undoStack.RemoveAll(x => x.HasTraceableObject == false);
-            undoStack.RemoveAll(x => x.Changes.Any(y => y.TraceableObject == traceable));
+            undoStack.RemoveAll(x => x.Changes.Any(y => traceables.Contains(y.TraceableObject)));
             redoStack.RemoveAll(x => x.HasTraceableObject == false);
-            redoStack.RemoveAll(x => x.Changes.Any(y => y.TraceableObject == traceable));
+            redoStack.RemoveAll(x => x.Changes.Any(y => traceables.Contains(y.TraceableObject)));
             TracedObjectHistoryChanged?.Invoke(this, EventArgs.Empty);
         }