From 8f4417cf61c92cf400d50f6b4bb05022be7882b6 Mon Sep 17 00:00:00 2001
From: Sergio Pedri <sergio0694@live.com>
Date: Thu, 13 Aug 2020 16:30:27 +0200
Subject: [PATCH] Remove unnecessary indirection in ByteMemoryManager<T>

---
 src/ImageSharp/Memory/ByteMemoryManager{T}.cs | 16 +---------------
 1 file changed, 1 insertion(+), 15 deletions(-)

diff --git a/src/ImageSharp/Memory/ByteMemoryManager{T}.cs b/src/ImageSharp/Memory/ByteMemoryManager{T}.cs
index 3a9eb34c20..924230fc85 100644
--- a/src/ImageSharp/Memory/ByteMemoryManager{T}.cs
+++ b/src/ImageSharp/Memory/ByteMemoryManager{T}.cs
@@ -36,21 +36,7 @@ namespace SixLabors.ImageSharp.Memory
         /// <inheritdoc/>
         public override Span<T> GetSpan()
         {
-            if (MemoryMarshal.TryGetArray(this.memory, out ArraySegment<byte> arraySegment))
-            {
-                return MemoryMarshal.Cast<byte, T>(arraySegment.AsSpan());
-            }
-
-            if (MemoryMarshal.TryGetMemoryManager<byte, MemoryManager<byte>>(this.memory, out MemoryManager<byte> memoryManager))
-            {
-                return MemoryMarshal.Cast<byte, T>(memoryManager.GetSpan());
-            }
-
-            // This should never be reached, as Memory<T> can currently only be wrapping
-            // either a byte[] array or a MemoryManager<byte> instance in this case.
-            ThrowHelper.ThrowArgumentException("The input Memory<byte> instance was not valid.", nameof(this.memory));
-
-            return default;
+            return MemoryMarshal.Cast<byte, T>(this.memory.Span);
         }
 
         /// <inheritdoc/>