Merge pull request #667 from TheTrackerCouncil/reset-snes-connector

Force resetting of snes connector on restarting tracker

Author: MattEqualsCoder

src/TrackerCouncil.Smz3.Abstractions/TrackerBase.cs

@@ -8,7 +8,7 @@
 
 namespace TrackerCouncil.Smz3.Abstractions;
 
-public abstract class TrackerBase
+public abstract class TrackerBase : IDisposable
 {
     /// <summary>
     /// Occurs when any speech was recognized, regardless of configured
@@ -369,4 +369,6 @@ protected virtual void OnVoiceRecognitionEnabledChanged()
     {
         VoiceRecognitionEnabledChanged?.Invoke(this, EventArgs.Empty);
     }
+
+    public abstract void Dispose();
 }

src/TrackerCouncil.Smz3.Tracking/AutoTracking/AutoTracker.cs

@@ -53,6 +53,7 @@ public AutoTracker(ILogger<AutoTracker> logger,
         _config = randomizerConfigService;
         TrackerBase = trackerBase;
         _snesConnectorService = snesConnectorService;
+        _snesConnectorService.ClearRequests();
 
         foreach (var module in modules)
         {

src/TrackerCouncil.Smz3.Tracking/Tracker.cs

@@ -596,7 +596,7 @@ public override void Error()
     /// <summary>
     /// Cleans up resources used by this class.
     /// </summary>
-    public void Dispose()
+    public override void Dispose()
     {
         Dispose(disposing: true);
         GC.SuppressFinalize(this);
@@ -664,6 +664,7 @@ private void Dispose(bool disposing)
             {
                 (_recognizer as IDisposable)?.Dispose();
                 (_communicator as IDisposable)?.Dispose();
+                _moduleFactory.Dispose();
 
                 foreach (var timer in _idleTimers.Values)
                     timer.Dispose();

src/TrackerCouncil.Smz3.UI/Services/TrackerWindowService.cs

@@ -340,6 +340,7 @@ public async Task Shutdown()
         _trackerMapWindow?.Close();
         _trackerLocationsWindow?.Close();
         tracker.StopTracking();
+        tracker.Dispose();
         _dispatcherTimer.Stop();
         _window.Close();
     }