Avoid collection modification

dalyIsaac · May 18, 2024 · deb59c9 · deb59c9
1 parent f3628a4
commit deb59c9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/Whim/Store/MonitorSector/MonitorSector.cs b/src/Whim/Store/MonitorSector/MonitorSector.cs
@@ -31,8 +31,10 @@ public override void Initialize()
 
 	public override void DispatchEvents()
 	{
-		foreach (EventArgs eventArgs in _events)
+		// Use index access to prevent the list from being modified during enumeration.
+		for (int idx = 0; idx < _events.Count; idx++)
 		{
+			EventArgs eventArgs = _events[idx];
 			switch (eventArgs)
 			{
 				case MonitorsChangedEventArgs args:

diff --git a/src/Whim/Store/WindowSector/WindowSector.cs b/src/Whim/Store/WindowSector/WindowSector.cs
@@ -43,8 +43,10 @@ public override void Initialize()
 
 	public override void DispatchEvents()
 	{
-		foreach (EventArgs eventArgs in _events)
+		// Use index access to prevent the list from being modified during enumeration.
+		for (int idx = 0; idx < _events.Count; idx++)
 		{
+			EventArgs eventArgs = _events[idx];
 			switch (eventArgs)
 			{
 				case WindowAddedEventArgs args: