From ed0edfe20a62f3f3d15be52ad508d9915cf15e4b Mon Sep 17 00:00:00 2001
From: Poker <poker_sang@outlook.com>
Date: Mon, 7 Apr 2025 22:45:28 +0800
Subject: [PATCH] fix AniMetadata

---
 .../Formats/Ani/AniFrameMetadata.cs           | 10 +----
 src/ImageSharp/Formats/Ani/AniMetadata.cs     | 41 ++++++++++++++++---
 2 files changed, 36 insertions(+), 15 deletions(-)

diff --git a/src/ImageSharp/Formats/Ani/AniFrameMetadata.cs b/src/ImageSharp/Formats/Ani/AniFrameMetadata.cs
index 8a567c40a..928899476 100644
--- a/src/ImageSharp/Formats/Ani/AniFrameMetadata.cs
+++ b/src/ImageSharp/Formats/Ani/AniFrameMetadata.cs
@@ -53,15 +53,7 @@ public class AniFrameMetadata : IFormatFrameMetadata<AniFrameMetadata>
         };
 
     /// <inheritdoc/>
-    IDeepCloneable IDeepCloneable.DeepClone() => new AniFrameMetadata
-    {
-        FrameDelay = this.FrameDelay,
-        EncodingHeight = this.EncodingHeight,
-        EncodingWidth = this.EncodingWidth,
-        FrameCount = this.FrameCount
-
-        // TODO SubImageMetadata
-    };
+    IDeepCloneable IDeepCloneable.DeepClone() => this.DeepClone();
 
     /// <inheritdoc/>
     public FormatConnectingFrameMetadata ToFormatConnectingFrameMetadata() => new FormatConnectingFrameMetadata() { Duration = TimeSpan.FromSeconds(this.FrameDelay / 60d) };
diff --git a/src/ImageSharp/Formats/Ani/AniMetadata.cs b/src/ImageSharp/Formats/Ani/AniMetadata.cs
index cdebcbb24..9b30bedb6 100644
--- a/src/ImageSharp/Formats/Ani/AniMetadata.cs
+++ b/src/ImageSharp/Formats/Ani/AniMetadata.cs
@@ -21,21 +21,33 @@ public class AniMetadata : IFormatMetadata<AniMetadata>
     /// <summary>
     /// Gets or sets the width of frames in the animation.
     /// </summary>
+    /// <remarks>
+    /// Remains zero when <see cref="Flags"/> has flag <see cref="AniHeaderFlags.IsIcon"/>
+    /// </remarks>
     public uint Width { get; set; }
 
     /// <summary>
     /// Gets or sets the height of frames in the animation.
     /// </summary>
+    /// <remarks>
+    /// Remains zero when <see cref="Flags"/> has flag <see cref="AniHeaderFlags.IsIcon"/>
+    /// </remarks>
     public uint Height { get; set; }
 
     /// <summary>
     /// Gets or sets the number of bits per pixel.
     /// </summary>
+    /// <remarks>
+    /// Remains zero when <see cref="Flags"/> has flag <see cref="AniHeaderFlags.IsIcon"/>
+    /// </remarks>
     public uint BitCount { get; set; }
 
     /// <summary>
     /// Gets or sets the number of frames in the animation.
     /// </summary>
+    /// <remarks>
+    /// Remains zero when <see cref="Flags"/> has flag <see cref="AniHeaderFlags.IsIcon"/>
+    /// </remarks>
     public uint Planes { get; set; }
 
     /// <summary>
@@ -64,21 +76,38 @@ public class AniMetadata : IFormatMetadata<AniMetadata>
     public IList<ImageFrameMetadata> IconFrames { get; set; } = [];
 
     /// <inheritdoc/>
-    public static AniMetadata FromFormatConnectingMetadata(FormatConnectingMetadata metadata) => throw new NotImplementedException();
+    public static AniMetadata FromFormatConnectingMetadata(FormatConnectingMetadata metadata)
+        => throw new NotImplementedException();
 
     /// <inheritdoc/>
     public void AfterImageApply<TPixel>(Image<TPixel> destination)
-        where TPixel : unmanaged, IPixel<TPixel> => throw new NotImplementedException();
+        where TPixel : unmanaged, IPixel<TPixel>
+    {
+    }
 
     /// <inheritdoc/>
-    public IDeepCloneable DeepClone() => throw new NotImplementedException();
+    IDeepCloneable IDeepCloneable.DeepClone() => this.DeepClone();
 
     /// <inheritdoc/>
-    public PixelTypeInfo GetPixelTypeInfo() => throw new NotImplementedException();
+    public PixelTypeInfo GetPixelTypeInfo()
+        => throw new NotImplementedException();
 
     /// <inheritdoc/>
-    public FormatConnectingMetadata ToFormatConnectingMetadata() => throw new NotImplementedException();
+    public FormatConnectingMetadata ToFormatConnectingMetadata()
+        => throw new NotImplementedException();
 
     /// <inheritdoc/>
-    AniMetadata IDeepCloneable<AniMetadata>.DeepClone() => throw new NotImplementedException();
+    public AniMetadata DeepClone() => new()
+    {
+        Width = this.Width,
+        Height = this.Height,
+        BitCount = this.BitCount,
+        Planes = this.Planes,
+        DisplayRate = this.DisplayRate,
+        Flags = this.Flags,
+        Name = this.Name,
+        Artist = this.Artist
+
+        // TODO IconFrames
+    };
 }