Minor cleanup and comments

Author: JimBobSquarePants

samples/DrawShapesWithImageSharp/Program.cs

@@ -55,8 +55,7 @@ private static void OutputStars()
 
             DrawText(
                 "Hello World Hello World Hello World Hello World Hello World Hello World Hello World",
-                 new EllipsePolygon(PointF.Empty, 100));
-            // new RectangularPolygon(PointF.Empty, new SizeF(100, 100)));
+                new EllipsePolygon(PointF.Empty, 100));
         }
 
         private static void DrawText(string text)

src/ImageSharp.Drawing/Processing/Processors/Text/DrawTextProcessor.cs

@@ -2,7 +2,6 @@
 // Licensed under the Apache License, Version 2.0.
 
 using System;
-using SixLabors.Fonts;
 using SixLabors.ImageSharp.PixelFormats;
 using SixLabors.ImageSharp.Processing.Processors;
 
@@ -47,7 +46,7 @@ public DrawTextProcessor(DrawingOptions drawingOptions, TextDrawingOptions textO
         public DrawingOptions DrawingOptions { get; }
 
         /// <summary>
-        /// Gets the <see cref="Fonts.TextDrawingOptions"/> defining text-specific drawing settings.
+        /// Gets the <see cref="Processing.TextDrawingOptions"/> defining text-specific drawing settings.
         /// </summary>
         public TextDrawingOptions TextOptions { get; }
 

src/ImageSharp.Drawing/Processing/Processors/Text/DrawTextProcessor{TPixel}.cs

@@ -152,10 +152,10 @@ private struct TextDecorationDetails
             public float Thickness { get; internal set; }
         }
 
+        // TODO: Why doesn't this inherit BaseGlyphBuilder?
+        // Ideally we should look to making text drawing capable of rendering rich text along a path.
         private class CachingGlyphRenderer : IColorGlyphRenderer, IGlyphDecorationRenderer, IDisposable
         {
-
-
             // just enough accuracy to allow for 1/8 pixel differences which
             // later are accumulated while rendering, but do not grow into full pixel offsets
             // The value 8 is benchmarked to:
@@ -273,7 +273,7 @@ public void SetDecoration(TextDecorations textDecorations, Vector2 start, Vector
                 }
 
                 // TODO:Isn't this already handled in font via GetEnds?
-                //fix up the thickness/Y position so that the line render nicly
+                // fix up the thickness/Y position so that the line render nicly
                 var thicknessOffset = new Vector2(0, thickness * .5f);
                 var tl = start - thicknessOffset;
                 var bl = start + thicknessOffset;
@@ -613,8 +613,6 @@ public void QuadraticBezierTo(Vector2 secondControlPoint, Vector2 point)
                 this.currentPoint = point;
             }
 
-
-
             private struct GlyphRenderData : IDisposable
             {
                 public Color? Color;

src/ImageSharp.Drawing/Processing/SolidPen.cs

@@ -1,8 +1,6 @@
 // Copyright (c) Six Labors.
 // Licensed under the Apache License, Version 2.0.
 
-using System.Diagnostics.CodeAnalysis;
-
 namespace SixLabors.ImageSharp.Drawing.Processing
 {
     /// <summary>

src/ImageSharp.Drawing/Processing/TextDrawingOptions.cs

@@ -0,0 +1,42 @@
+// Copyright (c) Six Labors.
+// Licensed under the Apache License, Version 2.0.
+
+using System.Collections.Generic;
+using SixLabors.Fonts;
+
+namespace SixLabors.ImageSharp.Drawing.Processing
+{
+    /// <summary>
+    /// Provides configuration options for rendering and shaping of drawable text.
+    /// </summary>
+    public class TextDrawingOptions : TextOptions
+    {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="TextDrawingOptions" /> class.
+        /// </summary>
+        /// <param name="font">The font.</param>
+        public TextDrawingOptions(Font font)
+            : base(font)
+        {
+        }
+
+        /// <summary>
+        /// Initializes a new instance of the <see cref="TextDrawingOptions" /> class from properties
+        /// copied from the given instance.
+        /// </summary>
+        /// <param name="options">The options whose properties are copied into this instance.</param>
+        public TextDrawingOptions(TextDrawingOptions options)
+            : base(options)
+        {
+        }
+
+        /// <summary>
+        /// Gets or sets an optional collection of text runs to apply to the body of text.
+        /// </summary>
+        public new IReadOnlyList<TextDrawingRun> TextRuns
+        {
+            get => (IReadOnlyList<TextDrawingRun>)base.TextRuns;
+            set => base.TextRuns = value;
+        }
+    }
+}

src/ImageSharp.Drawing/Processing/TextDrawingRun.cs

@@ -1,47 +1,38 @@
 // Copyright (c) Six Labors.
 // Licensed under the Apache License, Version 2.0.
 
-using System;
-using System.Collections.Generic;
-using System.Text;
 using SixLabors.Fonts;
-using SixLabors.ImageSharp.Drawing.Processing;
 
-namespace SixLabors.Fonts
+namespace SixLabors.ImageSharp.Drawing.Processing
 {
-    public class TextDrawingOptions : TextOptions
-    {
-        public TextDrawingOptions(Font font)
-            : base(font)
-        {
-        }
-
-        public TextDrawingOptions(TextDrawingOptions options)
-            : base(options)
-        {
-        }
-
-        public new IReadOnlyList<TextDrawingRun> TextRuns
-        {
-            get => (IReadOnlyList<TextDrawingRun>)base.TextRuns;
-            set => base.TextRuns = value;
-        }
-    }
-
+    /// <summary>
+    /// Represents a run of drawable text spanning a series of graphemes within a string.
+    /// </summary>
     public class TextDrawingRun : TextRun
     {
+        /// <summary>
+        /// Gets or sets the brush used for filling this run.
+        /// </summary>
         public IBrush Brush { get; set; }
 
+        /// <summary>
+        /// Gets or sets the pen used for outlining this run.
+        /// </summary>
         public IPen Pen { get; set; }
 
+        /// <summary>
+        /// Gets or sets the pen used for drawing strikeout features for this run.
+        /// </summary>
         public IPen StrikeoutPen { get; set; }
 
+        /// <summary>
+        /// Gets or sets the pen used for drawing underline features for this run.
+        /// </summary>
         public IPen UnderlinePen { get; set; }
 
+        /// <summary>
+        /// Gets or sets the pen used for drawing overline features for this run.
+        /// </summary>
         public IPen OverlinePen { get; set; }
-
-        public TextDrawingRun()
-        {
-        }
     }
 }

src/ImageSharp.Drawing/Shapes/Text/TextBuilder.cs

@@ -2,6 +2,7 @@
 // Licensed under the Apache License, Version 2.0.
 
 using SixLabors.Fonts;
+using SixLabors.ImageSharp.Drawing.Processing;
 using SixLabors.ImageSharp.Drawing.Text;
 
 namespace SixLabors.ImageSharp.Drawing

tests/ImageSharp.Drawing.Benchmarks/Drawing/DrawText.cs

@@ -5,7 +5,6 @@
 using System.Drawing.Drawing2D;
 using System.Linq;
 using BenchmarkDotNet.Attributes;
-using SixLabors.Fonts;
 using SixLabors.ImageSharp.Drawing.Processing;
 using SixLabors.ImageSharp.PixelFormats;
 using SixLabors.ImageSharp.Processing;

tests/ImageSharp.Drawing.Benchmarks/Drawing/DrawTextOutline.cs

@@ -5,7 +5,6 @@
 using System.Drawing.Drawing2D;
 using System.Linq;
 using BenchmarkDotNet.Attributes;
-using SixLabors.Fonts;
 using SixLabors.ImageSharp.Drawing.Processing;
 using SixLabors.ImageSharp.PixelFormats;
 using SixLabors.ImageSharp.Processing;