fix(app-launch): Dispose of _scavengeTimer to prevent resource leaks in ApplicationLaunchMonitor

carldebilly · carldebilly · commit 3c1b55ebe619 · 2025-10-10T13:44:04.000-04:00
diff --git a/src/Uno.UI.RemoteControl.Server/AppLaunch/ApplicationLaunchMonitor.cs b/src/Uno.UI.RemoteControl.Server/AppLaunch/ApplicationLaunchMonitor.cs
@@ -2,7 +2,6 @@
 using System.Collections.Concurrent;
 using System.Collections.Generic;
 using System.Threading;
-using System.Threading.Tasks;
 
 namespace Uno.UI.RemoteControl.Server.AppLaunch;
 
@@ -290,6 +289,8 @@ private void RunScavengePass()
 	/// </summary>
 	public void Dispose()
 	{
+		_scavengeTimer?.Dispose();
+
 		// Dispose all timers and clear pending timeout tasks
 		foreach (var kvp in _timeoutTasks.ToArray())
 		{

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,6 @@`
`2`	`2`	`using System.Collections.Concurrent;`
`3`	`3`	`using System.Collections.Generic;`
`4`	`4`	`using System.Threading;`
`5`		`-using System.Threading.Tasks;`
`6`	`5`
`7`	`6`	`namespace Uno.UI.RemoteControl.Server.AppLaunch;`
`8`	`7`
`@@ -290,6 +289,8 @@ private void RunScavengePass()`
`290`	`289`	`/// </summary>`
`291`	`290`	`public void Dispose()`
`292`	`291`	`{`
	`292`	`+ _scavengeTimer?.Dispose();`
	`293`	`+`
`293`	`294`	`// Dispose all timers and clear pending timeout tasks`
`294`	`295`	`foreach (var kvp in _timeoutTasks.ToArray())`
`295`	`296`	`{`