Merge pull request #1218 from b-editor/implement-gethashcode-for-medi…

…a-sources feat: implement GetHashCode for VideoSource, SoundSource, and BitmapSource
b-editor · Dec 28, 2024 · c5edf55 · c5edf55
2 parents 4a57d3f + f848398
commit c5edf55
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 0 deletions.
diff --git a/src/Beutl.Engine/Media/Source/BitmapSource.cs b/src/Beutl.Engine/Media/Source/BitmapSource.cs
@@ -83,4 +83,10 @@ public override bool Equals(object? obj)
                && !IsDisposed && !source.IsDisposed
                && ReferenceEquals(_bitmap.Value, source._bitmap.Value);
     }
+
+    public override int GetHashCode()
+    {
+        // ReSharper disable once NonReadonlyMemberInGetHashCode
+        return HashCode.Combine(!IsDisposed ? _bitmap.Value : null);
+    }
 }
diff --git a/src/Beutl.Engine/Media/Source/SoundSource.cs b/src/Beutl.Engine/Media/Source/SoundSource.cs
@@ -114,6 +114,12 @@ public override bool Equals(object? obj)
                && ReferenceEquals(_mediaReader.Value, source._mediaReader.Value);
     }
 
+    public override int GetHashCode()
+    {
+        // ReSharper disable once NonReadonlyMemberInGetHashCode
+        return HashCode.Combine(!IsDisposed ? _mediaReader.Value : null);
+    }
+
     private int ToSamples(TimeSpan timeSpan)
     {
         return (int)(timeSpan.TotalSeconds * SampleRate);

diff --git a/src/Beutl.Engine/Media/Source/VideoSource.cs b/src/Beutl.Engine/Media/Source/VideoSource.cs
@@ -100,6 +100,12 @@ public override bool Equals(object? obj)
                && ReferenceEquals(_mediaReader.Value, source._mediaReader.Value);
     }
 
+    public override int GetHashCode()
+    {
+        // ReSharper disable once NonReadonlyMemberInGetHashCode
+        return HashCode.Combine(!IsDisposed ? _mediaReader.Value : null);
+    }
+
     IVideoSource IVideoSource.Clone() => Clone();
 
     IMediaSource IMediaSource.Clone() => Clone();