Merge pull request #274 from SixLabors/js/fix-241

Add tests for PatterBrush issue 241

Author: JimBobSquarePants

src/ImageSharp.Drawing/Processing/PatternBrush.cs

@@ -4,7 +4,6 @@
 using System;
 using System.Numerics;
 using SixLabors.ImageSharp.Drawing.Utilities;
-using SixLabors.ImageSharp.Memory;
 using SixLabors.ImageSharp.PixelFormats;
 
 namespace SixLabors.ImageSharp.Drawing.Processing
@@ -118,8 +117,6 @@ private sealed class PatternBrushApplicator<TPixel> : BrushApplicator<TPixel>
             where TPixel : unmanaged, IPixel<TPixel>
         {
             private readonly DenseMatrix<TPixel> pattern;
-            private readonly MemoryAllocator allocator;
-            private readonly int scalineWidth;
             private readonly ThreadLocalBlenderBuffers<TPixel> blenderBuffers;
             private bool isDisposed;
 
@@ -138,8 +135,6 @@ public PatternBrushApplicator(
                 : base(configuration, options, source)
             {
                 this.pattern = pattern;
-                this.scalineWidth = source.Width;
-                this.allocator = configuration.MemoryAllocator;
                 this.blenderBuffers = new ThreadLocalBlenderBuffers<TPixel>(configuration.MemoryAllocator, source.Width);
             }
 

tests/ImageSharp.Drawing.Tests/Issues/Issue_241.cs

@@ -0,0 +1,34 @@
+// Copyright (c) Six Labors.
+// Licensed under the Apache License, Version 2.0.
+
+using System.Numerics;
+using SixLabors.Fonts;
+using SixLabors.ImageSharp.Drawing.Processing;
+using SixLabors.ImageSharp.PixelFormats;
+using SixLabors.ImageSharp.Processing;
+using Xunit;
+
+namespace SixLabors.ImageSharp.Drawing.Tests.Issues
+{
+    public class Issue_241
+    {
+        [Fact]
+        public void DoesNotThrowArgumentOutOfRangeException()
+        {
+            if (!TestEnvironment.IsWindows)
+            {
+                return;
+            }
+
+            FontFamily fontFamily = SystemFonts.Get("Arial");
+            RichTextOptions opt = new(fontFamily.CreateFont(100, FontStyle.Regular))
+            {
+                Origin = new Vector2(159, 0)
+            };
+            const string content = "TEST";
+
+            using Image image = new Image<Rgba32>(512, 256, Color.Black);
+            image.Mutate(x => x.DrawText(opt, content, Brushes.Horizontal(Color.Orange)));
+        }
+    }
+}