diff --git a/src/ImageSharp/IImage.cs b/src/ImageSharp/IImage.cs
index b9e2cee61..0d4dc3c9d 100644
--- a/src/ImageSharp/IImage.cs
+++ b/src/ImageSharp/IImage.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Six Labors and contributors.
+// Copyright (c) Six Labors and contributors.
// Licensed under the Apache License, Version 2.0.
using System;
@@ -11,4 +11,4 @@ namespace SixLabors.ImageSharp
public interface IImage : IImageInfo, IDisposable
{
}
-}
\ No newline at end of file
+}
diff --git a/src/ImageSharp/Image.cs b/src/ImageSharp/Image.cs
index 57f60f2e7..696f83662 100644
--- a/src/ImageSharp/Image.cs
+++ b/src/ImageSharp/Image.cs
@@ -1,6 +1,7 @@
// Copyright (c) Six Labors and contributors.
// Licensed under the Apache License, Version 2.0.
+using System;
using System.IO;
using SixLabors.ImageSharp.Advanced;
@@ -80,21 +81,11 @@ namespace SixLabors.ImageSharp
///
Configuration IConfigurable.Configuration => this.Configuration;
- ///
- /// Gets a value indicating whether the image instance is disposed.
- ///
- public bool IsDisposed { get; private set; }
-
///
public void Dispose()
{
- if (this.IsDisposed)
- {
- return;
- }
-
- this.IsDisposed = true;
- this.DisposeImpl();
+ this.Dispose(true);
+ GC.SuppressFinalize(this);
}
///
@@ -109,7 +100,7 @@ namespace SixLabors.ImageSharp
Guard.NotNull(encoder, nameof(encoder));
this.EnsureNotDisposed();
- EncodeVisitor visitor = new EncodeVisitor(encoder, stream);
+ var visitor = new EncodeVisitor(encoder, stream);
this.AcceptVisitor(visitor);
}
@@ -144,9 +135,15 @@ namespace SixLabors.ImageSharp
protected void UpdateSize(Size size) => this.size = size;
///
- /// Implements the Dispose logic.
+ /// Disposes the object and frees resources for the Garbage Collector.
+ ///
+ /// Whether to dispose of managed and unmanaged objects.
+ protected abstract void Dispose(bool disposing);
+
+ ///
+ /// Throws if the image is disposed.
///
- protected abstract void DisposeImpl();
+ internal abstract void EnsureNotDisposed();
private class EncodeVisitor : IImageVisitor
{
diff --git a/src/ImageSharp/ImageExtensions.cs b/src/ImageSharp/ImageExtensions.cs
index 6ea2b234c..6cdc948d4 100644
--- a/src/ImageSharp/ImageExtensions.cs
+++ b/src/ImageSharp/ImageExtensions.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Six Labors and contributors.
+// Copyright (c) Six Labors and contributors.
// Licensed under the Apache License, Version 2.0.
using System;
@@ -119,16 +119,5 @@ namespace SixLabors.ImageSharp
return $"data:{format.DefaultMimeType};base64,{Convert.ToBase64String(stream.ToArray())}";
}
}
-
- ///
- /// Throws if the image is disposed.
- ///
- internal static void EnsureNotDisposed(this Image image)
- {
- if (image.IsDisposed)
- {
- throw new ObjectDisposedException(nameof(image), "Trying to execute an operation on a disposed image.");
- }
- }
}
}
diff --git a/src/ImageSharp/ImageFrame.cs b/src/ImageSharp/ImageFrame.cs
index f3fe1ed8d..91872b21d 100644
--- a/src/ImageSharp/ImageFrame.cs
+++ b/src/ImageSharp/ImageFrame.cs
@@ -74,7 +74,17 @@ namespace SixLabors.ImageSharp
public Rectangle Bounds() => new Rectangle(0, 0, this.Width, this.Height);
///
- public abstract void Dispose();
+ public void Dispose()
+ {
+ this.Dispose(true);
+ GC.SuppressFinalize(this);
+ }
+
+ ///
+ /// Disposes the object and frees resources for the Garbage Collector.
+ ///
+ /// Whether to dispose of managed and unmanaged objects.
+ protected abstract void Dispose(bool disposing);
internal abstract void CopyPixelsTo(Span destination)
where TDestinationPixel : struct, IPixel;
diff --git a/src/ImageSharp/ImageFrame{TPixel}.cs b/src/ImageSharp/ImageFrame{TPixel}.cs
index 5c9ff489e..0436eb9d2 100644
--- a/src/ImageSharp/ImageFrame{TPixel}.cs
+++ b/src/ImageSharp/ImageFrame{TPixel}.cs
@@ -21,7 +21,7 @@ namespace SixLabors.ImageSharp
/// In all other cases it is the only frame of the image.
///
/// The pixel format.
- public sealed class ImageFrame : ImageFrame, IPixelSource, IDisposable
+ public sealed class ImageFrame : ImageFrame, IPixelSource
where TPixel : struct, IPixel
{
private bool isDisposed;
@@ -196,20 +196,20 @@ namespace SixLabors.ImageSharp
this.UpdateSize(this.PixelBuffer.Size());
}
- ///
- /// Disposes the object and frees resources for the Garbage Collector.
- ///
- public override void Dispose()
+ ///
+ protected override void Dispose(bool disposing)
{
if (this.isDisposed)
{
return;
}
- this.PixelBuffer?.Dispose();
- this.PixelBuffer = null;
+ if (disposing)
+ {
+ this.PixelBuffer?.Dispose();
+ this.PixelBuffer = null;
+ }
- // Note disposing is done.
this.isDisposed = true;
}
diff --git a/src/ImageSharp/Image{TPixel}.cs b/src/ImageSharp/Image{TPixel}.cs
index a7ea58652..181e818ee 100644
--- a/src/ImageSharp/Image{TPixel}.cs
+++ b/src/ImageSharp/Image{TPixel}.cs
@@ -1,4 +1,4 @@
-// Copyright (c) Six Labors and contributors.
+// Copyright (c) Six Labors and contributors.
// Licensed under the Apache License, Version 2.0.
using System;
@@ -21,6 +21,8 @@ namespace SixLabors.ImageSharp
public sealed class Image : Image
where TPixel : struct, IPixel
{
+ private bool isDisposed;
+
///
/// Initializes a new instance of the class
/// with the height and the width of the image.
@@ -185,7 +187,29 @@ namespace SixLabors.ImageSharp
}
///
- protected override void DisposeImpl() => this.Frames.Dispose();
+ protected override void Dispose(bool disposing)
+ {
+ if (this.isDisposed)
+ {
+ return;
+ }
+
+ if (disposing)
+ {
+ this.Frames.Dispose();
+ }
+
+ this.isDisposed = true;
+ }
+
+ ///
+ internal override void EnsureNotDisposed()
+ {
+ if (this.isDisposed)
+ {
+ throw new ObjectDisposedException("Trying to execute an operation on a disposed image.");
+ }
+ }
///
internal override void AcceptVisitor(IImageVisitor visitor)
diff --git a/src/ImageSharp/Processing/Processors/CloningImageProcessor{TPixel}.cs b/src/ImageSharp/Processing/Processors/CloningImageProcessor{TPixel}.cs
index 6b0329e77..1290a1032 100644
--- a/src/ImageSharp/Processing/Processors/CloningImageProcessor{TPixel}.cs
+++ b/src/ImageSharp/Processing/Processors/CloningImageProcessor{TPixel}.cs
@@ -17,8 +17,6 @@ namespace SixLabors.ImageSharp.Processing.Processors
public abstract class CloningImageProcessor : ICloningImageProcessor
where TPixel : struct, IPixel
{
- private bool isDisposed;
-
///
/// Initializes a new instance of the class.
///
@@ -109,6 +107,7 @@ namespace SixLabors.ImageSharp.Processing.Processors
public void Dispose()
{
this.Dispose(true);
+ GC.SuppressFinalize(this);
}
///
@@ -165,10 +164,6 @@ namespace SixLabors.ImageSharp.Processing.Processors
/// Whether to dispose managed and unmanaged objects.
protected virtual void Dispose(bool disposing)
{
- if (!this.isDisposed)
- {
- this.isDisposed = true;
- }
}
private void CheckFrameCount(Image a, Image b)
diff --git a/src/ImageSharp/Processing/Processors/ImageProcessor{TPixel}.cs b/src/ImageSharp/Processing/Processors/ImageProcessor{TPixel}.cs
index b224adc3f..eb1dc4ba0 100644
--- a/src/ImageSharp/Processing/Processors/ImageProcessor{TPixel}.cs
+++ b/src/ImageSharp/Processing/Processors/ImageProcessor{TPixel}.cs
@@ -16,8 +16,6 @@ namespace SixLabors.ImageSharp.Processing.Processors
public abstract class ImageProcessor : IImageProcessor
where TPixel : struct, IPixel
{
- private bool isDisposed;
-
///
/// Initializes a new instance of the class.
///
@@ -98,6 +96,8 @@ namespace SixLabors.ImageSharp.Processing.Processors
///
public virtual void Dispose()
{
+ this.Dispose(true);
+ GC.SuppressFinalize(this);
}
///
@@ -143,10 +143,6 @@ namespace SixLabors.ImageSharp.Processing.Processors
/// Whether to dispose managed and unmanaged objects.
protected virtual void Dispose(bool disposing)
{
- if (!this.isDisposed)
- {
- this.isDisposed = true;
- }
}
}
}