Remove NormalizeOutput

Author: JimBobSquarePants

src/ImageSharp.Drawing/PolygonGeometry/StrokedShapeGenerator.cs

@@ -146,7 +146,6 @@ private static PolygonClipper.StrokeOptions CreateStrokeOptions(StrokeOptions op
             ArcDetailScale = options.ArcDetailScale,
             MiterLimit = options.MiterLimit,
             InnerMiterLimit = options.InnerMiterLimit,
-            NormalizeOutput = options.NormalizeOutput,
             LineJoin = options.LineJoin switch
             {
                 LineJoin.MiterRound => PolygonClipper.LineJoin.MiterRound,

src/ImageSharp.Drawing/Processing/DrawingCanvas{TPixel}.cs

@@ -469,9 +469,8 @@ public void Draw(Pen pen, IPath path)
 
         IPath outline = pen.GeneratePath(transformedPath);
 
-        // Non-normalized stroke output can self-overlap; non-zero winding preserves stroke semantics.
-        if (!pen.StrokeOptions.NormalizeOutput &&
-            effectiveOptions.ShapeOptions.IntersectionRule != IntersectionRule.NonZero)
+        // Stroke geometry can self-overlap; non-zero winding preserves stroke semantics.
+        if (effectiveOptions.ShapeOptions.IntersectionRule != IntersectionRule.NonZero)
         {
             ShapeOptions shapeOptions = effectiveOptions.ShapeOptions.DeepClone();
             shapeOptions.IntersectionRule = IntersectionRule.NonZero;
@@ -1038,9 +1037,8 @@ private CompositionCommand CreateCompositionCommand(
             compositionPath = pen.GeneratePath(operation.Path);
             samplingOrigin = RasterizerSamplingOrigin.PixelCenter;
 
-            // Keep draw semantics aligned with DrawPath: non-normalized stroke output
-            // requires non-zero winding to preserve stroke interior behavior.
-            if (!pen.StrokeOptions.NormalizeOutput && intersectionRule != IntersectionRule.NonZero)
+            // Stroke geometry can self-overlap; non-zero winding preserves stroke semantics.
+            if (intersectionRule != IntersectionRule.NonZero)
             {
                 intersectionRule = IntersectionRule.NonZero;
             }

src/ImageSharp.Drawing/Processing/StrokeOptions.cs

@@ -8,16 +8,6 @@ namespace SixLabors.ImageSharp.Drawing.Processing;
 /// </summary>
 public sealed class StrokeOptions : IEquatable<StrokeOptions?>
 {
-    /// <summary>
-    /// Gets or sets a value indicating whether stroked contours should be normalized by
-    /// resolving self-intersections and overlaps before returning.
-    /// </summary>
-    /// <remarks>
-    /// Defaults to <see langword="false"/> for maximum throughput.
-    /// When disabled, callers should rasterize with a non-zero winding fill rule.
-    /// </remarks>
-    public bool NormalizeOutput { get; set; }
-
     /// <summary>
     /// Gets or sets the miter limit used to clamp outer miter joins.
     /// </summary>
@@ -56,7 +46,6 @@ public sealed class StrokeOptions : IEquatable<StrokeOptions?>
     /// <inheritdoc/>
     public bool Equals(StrokeOptions? other)
         => other is not null &&
-           this.NormalizeOutput == other.NormalizeOutput &&
            this.MiterLimit == other.MiterLimit &&
            this.InnerMiterLimit == other.InnerMiterLimit &&
            this.ArcDetailScale == other.ArcDetailScale &&
@@ -67,7 +56,6 @@ public bool Equals(StrokeOptions? other)
     /// <inheritdoc/>
     public override int GetHashCode()
         => HashCode.Combine(
-            this.NormalizeOutput,
             this.MiterLimit,
             this.InnerMiterLimit,
             this.ArcDetailScale,

tests/ImageSharp.Drawing.Benchmarks/Drawing/DrawPolygon.cs

@@ -208,19 +208,7 @@ public void ImageSharpCombinedPathsWebGPUNativeSurface()
     }
 
     [Benchmark]
-    public IPath ImageSharpStrokeAndClipCombined() => this.isPen.GeneratePath(this.imageSharpPath);
-
-    [Benchmark]
-    public IPath ImageSharpStrokeAndClipSeparate()
-    {
-        IPath path = Path.Empty;
-        foreach (PointF[] loop in this.points)
-        {
-            path = this.isPen.GeneratePath(new Polygon(loop));
-        }
-
-        return path;
-    }
+    public IPath ImageSharpStroke() => this.isPen.GeneratePath(this.imageSharpPath);
 
     [Benchmark]
     public void FillPolygon()

tests/ImageSharp.Drawing.Tests/Processing/DrawingCanvasTests.StrokeOptions.cs

@@ -20,15 +20,9 @@ public void Draw_NormalizeOutputFalse_MatchesReference<TPixel>(TestImageProvider
         IPath leftPath = CreateBowTiePath(new RectangleF(28, 34, 128, 152));
         IPath rightPath = CreateBowTiePath(new RectangleF(204, 34, 128, 152));
 
-        SolidPen nonNormalizedPen = new(Color.CornflowerBlue.WithAlpha(0.88F), 24F);
-        nonNormalizedPen.StrokeOptions.NormalizeOutput = false;
-        nonNormalizedPen.StrokeOptions.LineJoin = LineJoin.Round;
-        nonNormalizedPen.StrokeOptions.LineCap = LineCap.Round;
-
-        SolidPen normalizedPen = new(Color.CornflowerBlue.WithAlpha(0.88F), 24F);
-        normalizedPen.StrokeOptions.NormalizeOutput = true;
-        normalizedPen.StrokeOptions.LineJoin = LineJoin.Round;
-        normalizedPen.StrokeOptions.LineCap = LineCap.Round;
+        SolidPen pen = new(Color.CornflowerBlue.WithAlpha(0.88F), 24F);
+        pen.StrokeOptions.LineJoin = LineJoin.Round;
+        pen.StrokeOptions.LineCap = LineCap.Round;
 
         DrawingOptions evenOddOptions = new()
         {
@@ -39,8 +33,8 @@ public void Draw_NormalizeOutputFalse_MatchesReference<TPixel>(TestImageProvider
         canvas.Fill(new Rectangle(12, 12, 336, 196), Brushes.Solid(Color.GhostWhite.WithAlpha(0.85F)));
 
         _ = canvas.Save(evenOddOptions);
-        canvas.Draw(nonNormalizedPen, leftPath);
-        canvas.Draw(normalizedPen, rightPath);
+        canvas.Draw(pen, leftPath);
+        canvas.Draw(pen, rightPath);
         canvas.Restore();
 
         canvas.Draw(Pens.Solid(Color.DarkSlateGray, 2F), leftPath);