Bug fix: exception need throw

dotnet · May 29, 2021 · 26b348e · 26b348e
1 parent 7c34ab9
commit 26b348e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/Microsoft.ML.TensorFlow/TensorflowTransform.cs b/src/Microsoft.ML.TensorFlow/TensorflowTransform.cs
@@ -354,7 +354,7 @@ internal static (TF_DataType[] tfInputTypes, TensorShape[] tfInputShapes, (Opera
         internal static TensorShape GetTensorShape(TF_Output output, Graph graph, Status status = null)
         {
             if (graph == IntPtr.Zero)
-                new ObjectDisposedException(nameof(graph));
+                throw new ObjectDisposedException(nameof(graph));
 
             var cstatus = status == null ? new Status() : status;
             var n = c_api.TF_GraphGetTensorNumDims(graph, output, cstatus.Handle);

diff --git a/src/Microsoft.ML.TensorFlow/TensorflowUtils.cs b/src/Microsoft.ML.TensorFlow/TensorflowUtils.cs
@@ -584,7 +584,7 @@ public static TF_Output ParseOutput(Session session, string operation)
             public Tensor[] Run()
             {
                 if (_session == IntPtr.Zero)
-                    new ObjectDisposedException(nameof(_session));
+                    throw new ObjectDisposedException(nameof(_session));
 
                 unsafe
                 {

diff --git a/src/Microsoft.ML.Vision/DnnRetrainTransform.cs b/src/Microsoft.ML.Vision/DnnRetrainTransform.cs
@@ -605,7 +605,7 @@ internal static (TF_DataType[] tfInputTypes, TensorShape[] tfInputShapes, (Opera
         internal static TensorShape GetTensorShape(TF_Output output, Graph graph, Status status = null)
         {
             if (graph == IntPtr.Zero)
-                new ObjectDisposedException(nameof(graph));
+                throw new ObjectDisposedException(nameof(graph));
 
             var cstatus = status == null ? new Status() : status;
             var n = c_api.TF_GraphGetTensorNumDims(graph, output, cstatus.Handle);