Merge branch 'step-2' into develop

Author: kekyo

FlashCap.Core/Devices/AVFoundationDevice.cs

@@ -42,23 +42,36 @@ public AVFoundationDevice(string uniqueID, string modelID) :
     }
 
     protected override async Task OnDisposeAsync()
-    {
-        if (this.session != null)
-        {
-            this.session.StopRunning();
-            this.session.Dispose();
+    {
+        // Ensure that we stop the session if it's running
+        if (this.session is not null && IsRunning)
+        {
+            this.session.StopRunning();
+            IsRunning = false;
         }
-        
-        this.device?.Dispose();
+
+        this.session?.Dispose();
         this.deviceInput?.Dispose();
         this.deviceOutput?.Dispose();
+        this.device?.Dispose();
         this.queue?.Dispose();
 
-        Marshal.FreeHGlobal(this.bitmapHeader);
+        this.session = null;
+        this.deviceInput = null;
+        this.deviceOutput = null;
+        this.device = null;
+        this.queue = null;
+
+        if (this.bitmapHeader != IntPtr.Zero)
+        {
+            Marshal.FreeHGlobal(this.bitmapHeader);
+            this.bitmapHeader = IntPtr.Zero;
+        }
 
-        if (frameProcessor is not null)
+        if (this.frameProcessor is not null)
         {
-            await frameProcessor.DisposeAsync().ConfigureAwait(false);
+            await this.frameProcessor.DisposeAsync().ConfigureAwait(false);
+            this.frameProcessor = null;
         }
 
         await base.OnDisposeAsync().ConfigureAwait(false);

FlashCap.Core/Internal/AVFoundation/AVCaptureVideoDataOutput.cs

@@ -73,7 +73,7 @@ public void SetPixelFormatType(int format)
             IntPtr pixelFormatTypeKeyPtr = Dlfcn.dlsym(LibCoreVideo.Handle, "kCVPixelBufferPixelFormatTypeKey");
             if (pixelFormatTypeKeyPtr == IntPtr.Zero)
             {
-                throw new Exception("Error comunicating with the AVCaptureVideoDataOutput");
+                throw new Exception("Error communicating with the AVCaptureVideoDataOutput");
             }
 
             // Get NSString value
@@ -107,7 +107,7 @@ public void SetSampleBufferDelegate(AVFoundationDevice.VideoBufferHandler sample
 
             IntPtr impCallback = Marshal.GetFunctionPointerForDelegate(callbackDelegate);
 
-            // "v@:@@@" this means the methood returns void and receives (self, _cmd, output, sampleBuffer, connection).
+            // "v@:@@@" this means the method returns void and receives (self, _cmd, output, sampleBuffer, connection).
             string types = "v@:@@@";
             bool added = LibObjC.class_addMethod(delegateClass, selDidOutput, impCallback, types);
             if (!added)