Some code cleanup

Author: ff-gerard

samples/Synercoding.FileFormats.Pdf.ConsoleTester/Program.cs

@@ -2,66 +2,65 @@
 using System;
 using System.Collections.Generic;
 
-namespace Synercoding.FileFormats.Pdf
+namespace Synercoding.FileFormats.Pdf;
+
+/// <summary>
+/// This class contains information about the document
+/// </summary>
+public class DocumentInformation
 {
-    /// <summary>
-    /// This class contains information about the document
-    /// </summary>
-    public class DocumentInformation
+    internal DocumentInformation(PdfReference id)
     {
-        internal DocumentInformation(PdfReference id)
-        {
-            Reference = id;
-        }
+        Reference = id;
+    }
 
-        /// <summary>
-        /// The document's title
-        /// </summary>
-        public string? Title { get; set; }
+    /// <summary>
+    /// The document's title
+    /// </summary>
+    public string? Title { get; set; }
 
-        /// <summary>
-        /// The name of the person who created the document
-        /// </summary>
-        public string? Author { get; set; }
+    /// <summary>
+    /// The name of the person who created the document
+    /// </summary>
+    public string? Author { get; set; }
 
-        /// <summary>
-        /// The subject of the document
-        /// </summary>
-        public string? Subject { get; set; }
+    /// <summary>
+    /// The subject of the document
+    /// </summary>
+    public string? Subject { get; set; }
 
-        /// <summary>
-        /// Keywords associated with the document
-        /// </summary>
-        public string? Keywords { get; set; }
+    /// <summary>
+    /// Keywords associated with the document
+    /// </summary>
+    public string? Keywords { get; set; }
 
-        /// <summary>
-        /// If the document was converted to PDF from another format, the name of the conforming product that created the original document from which it was converted. Otherwise the name of the application that created the document.
-        /// </summary>
-        public string? Creator { get; set; }
+    /// <summary>
+    /// If the document was converted to PDF from another format, the name of the conforming product that created the original document from which it was converted. Otherwise the name of the application that created the document.
+    /// </summary>
+    public string? Creator { get; set; }
 
-        /// <summary>
-        /// If the document was converted to PDF from another format, the name of the conforming product that converted it to PDF.
-        /// </summary>
-        public string? Producer { get; set; }
+    /// <summary>
+    /// If the document was converted to PDF from another format, the name of the conforming product that converted it to PDF.
+    /// </summary>
+    public string? Producer { get; set; }
 
-        /// <summary>
-        /// The date and time the document was created, in human-readable form.
-        /// </summary>
-        public DateTime? CreationDate { get; set; } = DateTime.Now;
+    /// <summary>
+    /// The date and time the document was created, in human-readable form.
+    /// </summary>
+    public DateTime? CreationDate { get; set; } = DateTime.Now;
 
-        /// <summary>
-        /// The date and time the document was most recently modified, in human-readable form.
-        /// </summary>
-        public DateTime? ModDate { get; set; }
+    /// <summary>
+    /// The date and time the document was most recently modified, in human-readable form.
+    /// </summary>
+    public DateTime? ModDate { get; set; }
 
-        /// <summary>
-        /// A pdf reference object that can be used to reference to this object
-        /// </summary>
-        public PdfReference Reference { get; }
+    /// <summary>
+    /// A pdf reference object that can be used to reference to this object
+    /// </summary>
+    public PdfReference Reference { get; }
 
-        /// <summary>
-        /// Extra information that will be added to the PDF meta data
-        /// </summary>
-        public IDictionary<string, string> ExtraInfo { get; } = new Dictionary<string, string>();
-    }
+    /// <summary>
+    /// Extra information that will be added to the PDF meta data
+    /// </summary>
+    public IDictionary<string, string> ExtraInfo { get; } = new Dictionary<string, string>();
 }

src/Synercoding.FileFormats.Pdf/DocumentInformation.cs

@@ -9,6 +9,6 @@ public static TContext WrapInState<TContext>(this TContext context, Action<TCont
         => context.WrapInState(contentOperations, static (operations, context) => operations(context));
 
     public static Task<TContext> WrapInStateAsync<TContext>(this TContext context, Func<TContext, Task> contentOperations)
-        where TContext: IContentContext<TContext>
+        where TContext : IContentContext<TContext>
         => context.WrapInStateAsync(contentOperations, static (operations, context) => operations(context));
 }

src/Synercoding.FileFormats.Pdf/Extensions/IContentContextExtensions.cs

@@ -87,10 +87,7 @@ public static IPageContentContext AddText(this IPageContentContext context, stri
         => context.AddText(text, font, size, ops => { });
 
     public static IPageContentContext AddText(this IPageContentContext context, string text, Font font, double size, Point location)
-        => context.AddText(text, font, size, location, static (location, ops) =>
-        {
-            ops.MoveToStartNextLine(location.X.AsRaw(Unit.Points), location.Y.AsRaw(Unit.Points));
-        });
+        => context.AddText(text, font, size, location, static (location, ops) => ops.MoveToStartNextLine(location.X.AsRaw(Unit.Points), location.Y.AsRaw(Unit.Points)));
 
     public static IPageContentContext AddText(this IPageContentContext context, string text, Font font, double size, Action<ITextContentContext> extraOperations)
         => context.AddText(text, font, size, extraOperations, static (extraOperations, context) => extraOperations(context));
@@ -113,13 +110,9 @@ public static IPageContentContext AddText<T>(this IPageContentContext context, s
             for (int i = 0; i < lines.Length; i++)
             {
                 if (i == 0)
-                {
                     context.ShowText(lines[i]);
-                }
                 else
-                {
                     context.ShowTextOnNextLine(lines[i]);
-                }
             }
         });
     }

src/Synercoding.FileFormats.Pdf/Extensions/IPageContentContextExtensions.cs

@@ -1,24 +1,23 @@
 using Synercoding.Primitives;
 
-namespace Synercoding.FileFormats.Pdf.Extensions
+namespace Synercoding.FileFormats.Pdf.Extensions;
+
+/// <summary>
+/// Extension methods for primitives
+/// </summary>
+public static class PrimitiveExtensions
 {
     /// <summary>
-    /// Extension methods for primitives
+    /// Convert a <see cref="Rectangle"/> to a transformation matrix
     /// </summary>
-    public static class PrimitiveExtensions
+    /// <param name="rectangle">The <see cref="Rectangle"/> to use</param>
+    /// <returns>Returns a <see cref="Matrix"/> representing the provided <see cref="Rectangle"/>.</returns>
+    public static Matrix AsPlacementMatrix(this Rectangle rectangle)
     {
-        /// <summary>
-        /// Convert a <see cref="Rectangle"/> to a transformation matrix
-        /// </summary>
-        /// <param name="rectangle">The <see cref="Rectangle"/> to use</param>
-        /// <returns>Returns a <see cref="Matrix"/> representing the provided <see cref="Rectangle"/>.</returns>
-        public static Matrix AsPlacementMatrix(this Rectangle rectangle)
-        {
-            rectangle = rectangle.ConvertTo(Unit.Points);
-            return new Matrix(
-                rectangle.URX.Raw - rectangle.LLX.Raw, 0,
-                0, rectangle.URY.Raw - rectangle.LLY.Raw,
-                rectangle.LLX.Raw, rectangle.LLY.Raw);
-        }
+        rectangle = rectangle.ConvertTo(Unit.Points);
+        return new Matrix(
+            rectangle.URX.Raw - rectangle.LLX.Raw, 0,
+            0, rectangle.URY.Raw - rectangle.LLY.Raw,
+            rectangle.LLX.Raw, rectangle.LLY.Raw);
     }
 }

src/Synercoding.FileFormats.Pdf/Extensions/PrimitiveExtensions.cs

@@ -1,5 +1,4 @@
 using Synercoding.FileFormats.Pdf.LowLevel;
-using Synercoding.Primitives;
 
 namespace Synercoding.FileFormats.Pdf;
 
@@ -11,7 +10,7 @@ public interface IShapeContext : IContentContext<IShapeContext>
     /// </summary>
     /// <param name="x">The X coordinate of the move</param>
     /// <param name="y">The Y coordinate of the move</param>
-    /// <returns>The calling <see cref="IPath"/> to support chaining operations.</returns>
+    /// <returns>The calling <see cref="IShapeContext"/> to support chaining operations.</returns>
     IShapeContext Move(double x, double y);