Move unused code to Unused internal namespace

Author: lilith

src/Imageflow/Fluent/BufferedStreamSource.cs

@@ -6,7 +6,7 @@ namespace Imageflow.Fluent;
 
 public sealed class BufferedStreamSource : IAsyncMemorySource, IMemorySource
 {
-    private BufferedStreamSource(Stream stream, bool disposeUnderlying, bool seekToStart = true)
+    private BufferedStreamSource(Stream stream, bool disposeUnderlying, bool seekToStart)
     {
         if (stream.Position != 0 && !stream.CanSeek && seekToStart)
         {
@@ -129,7 +129,7 @@ public static IAsyncMemorySource BorrowEntireStream(Stream stream)
     }
     /// <summary>
     /// <strong>You remain responsible for disposing and cleaning up the stream after the job is disposed.</strong>
-    /// Only reads from the current position to the end of the image file. 
+    /// Only reads from the current position to the end of the image file.
     /// You swear not to close, dispose, or reuse the stream or its underlying memory/stream until after this wrapper and the job are disposed.
     /// </summary>
     /// <param name="stream"></param>
@@ -154,8 +154,8 @@ public static IAsyncMemorySource UseEntireStreamAndDisposeWithSource(Stream stre
 
     /// <summary>
     /// The stream will be closed and disposed with the BufferedStreamSource.
-    /// <strong>You must not close, dispose, or reuse the stream or its underlying streams/buffers until after the job and the owning objects are disposed.</strong>
-    /// strong>The BufferedStreamSource will still need to be disposed after the job, either with a using declaration or by transferring ownership of it to the job (which should be in a using declaration).</strong>
+    /// <remarks>You must not close, dispose, or reuse the stream or its underlying streams/buffers until after the job and the owning objects are disposed.
+    /// The BufferedStreamSource will still need to be disposed after the job, either with a using declaration or by transferring ownership of it to the job (which should be in a using declaration).</remarks>
     /// </summary>
     /// <param name="stream"></param>
     /// <returns></returns>

src/Imageflow/Fluent/ImageJob.cs

@@ -300,40 +300,6 @@ internal async Task<BuildJobResult> FinishAsync(JobExecutionOptions executionOpt
         }
     }
 
-#pragma warning disable CS0618 // Type or member is obsolete
-    private class StreamJsonSpanProvider : IJsonResponseProvider, IJsonResponseSpanProvider, IJsonResponse
-#pragma warning restore CS0618 // Type or member is obsolete
-    {
-        private readonly MemoryStream _ms;
-
-        public StreamJsonSpanProvider(int statusCode, MemoryStream ms)
-        {
-            ImageflowErrorCode = statusCode;
-            _ms = ms;
-        }
-        public void Dispose() => _ms.Dispose();
-        public Stream GetStream() => _ms;
-        public ReadOnlySpan<byte> BorrowBytes()
-        {
-            return _ms.TryGetBufferSliceAllWrittenData(out var slice) ? slice : _ms.ToArray();
-        }
-
-        public int ImageflowErrorCode { get; }
-        public string CopyString()
-        {
-            return BorrowBytes().Utf8ToString();
-        }
-
-        public JsonNode? Parse()
-        {
-            return BorrowBytes().ParseJsonNode();
-        }
-
-        public byte[] CopyBytes()
-        {
-            return BorrowBytes().ToArray();
-        }
-    }
 
     // private object BuildJsonWithPlaceholders()
     // {
@@ -803,3 +769,4 @@ public static bool CanDecodeBytes(byte[] first12Bytes)
         return MagicBytes.GetImageContentType(first12Bytes);
     }
 }
+

src/Imageflow/Internal.Helpers/StreamMemoryExtensionsPolyfills.cs

@@ -1,5 +1,7 @@
+#if !NETSTANDARD2_1_OR_GREATER
 using System.Buffers;
 using System.Runtime.InteropServices;
+#endif
 
 namespace Imageflow.Internal.Helpers;
 

src/Imageflow/Internal.Helpers/OutputDestinationWrapper.cs renamed to src/Imageflow/Internal.Helpers/Unused/OutputDestinationWrapper.cs

@@ -1,7 +1,7 @@
 using System.Runtime.ConstrainedExecution;
 using System.Runtime.InteropServices;
 
-namespace Imageflow.Internal.Helpers;
+namespace Imageflow.Internal.Helpers.Unused;
 
 internal class PinBox : CriticalFinalizerObject, IDisposable
 {

src/Imageflow/Internal.Helpers/PinBox.cs renamed to src/Imageflow/Internal.Helpers/Unused/PinBox.cs

@@ -0,0 +1,39 @@
+using System.Text.Json.Nodes;
+using Imageflow.Bindings;
+
+namespace Imageflow.Internal.Helpers.Unused;
+
+#pragma warning disable CS0618
+internal class StreamJsonSpanProvider : IJsonResponseProvider, IJsonResponseSpanProvider, IJsonResponse
+#pragma warning restore CS0618 // Type or member is obsolete
+{
+    private readonly MemoryStream _ms;
+
+    public StreamJsonSpanProvider(int statusCode, MemoryStream ms)
+    {
+        ImageflowErrorCode = statusCode;
+        _ms = ms;
+    }
+    public void Dispose() => _ms.Dispose();
+    public Stream GetStream() => _ms;
+    public ReadOnlySpan<byte> BorrowBytes()
+    {
+        return _ms.TryGetBufferSliceAllWrittenData(out var slice) ? slice : _ms.ToArray();
+    }
+
+    public int ImageflowErrorCode { get; }
+    public string CopyString()
+    {
+        return BorrowBytes().Utf8ToString();
+    }
+
+    public JsonNode? Parse()
+    {
+        return BorrowBytes().ParseJsonNode();
+    }
+
+    public byte[] CopyBytes()
+    {
+        return BorrowBytes().ToArray();
+    }
+}

src/Imageflow/Internal.Helpers/Unused/StreamJsonSpanProvider.cs

@@ -1,7 +1,7 @@
 using System.Buffers;
 using System.Runtime.CompilerServices;
 
-namespace Imageflow.Internal.Helpers;
+namespace Imageflow.Internal.Helpers.Unused;
 
 internal unsafe class UnmanagedMemoryManager : MemoryManager<byte>
 {

src/Imageflow/Internal.Helpers/UnmanagedMemoryManager.cs renamed to src/Imageflow/Internal.Helpers/Unused/UnmanagedMemoryManager.cs

@@ -8,8 +8,9 @@
 
 using Xunit;
 using Xunit.Abstractions;
-
+#if NET8_0_OR_GREATER
 using JsonNamingPolicy = System.Text.Json.JsonNamingPolicy;
+#endif
 using JsonSerializerOptions = System.Text.Json.JsonSerializerOptions;
 #pragma warning disable CS0618 // Type or member is obsolete