DisposableObject #362

src/TensorFlowNET.Core/DisposableObject.cs

@@ -34,19 +34,19 @@ namespace Tensorflow
 
         private void internal_dispose(bool disposing)
         {
-            // free unmanaged resources (unmanaged objects).
-            if (_handle != IntPtr.Zero)
+            if (disposing)
             {
-                // set large fields to null.
-                DisposeUnmanagedResources(_handle);
+                // free unmanaged resources (unmanaged objects) and override a finalizer below.
+                if (_handle != IntPtr.Zero)
+                {
+                    // dispose managed state (managed objects).
+                    DisposeManagedResources();
 
-                _handle = IntPtr.Zero;
-            }
+                    // set large fields to null.
+                    DisposeUnmanagedResources(_handle);
 
-            if (disposing)
-            {
-                // dispose managed state (managed objects).
-                DisposeManagedResources();
+                    _handle = IntPtr.Zero;
+                }
             }
         }
 

src/TensorFlowNET.Core/Sessions/BaseSession.cs

@@ -49,7 +49,7 @@ namespace Tensorflow
 
             // dispose newOpts
             if (opts == null)
-                c_api.TF_DeleteSessionOptions(newOpts);
+                newOpts.Dispose();
 
             status.Check(true);
         }