🚧Refactor GlyphMetrics to readonly record (#820)

* Start work for issue #803 * Refactor: GlyphMetrics struct to readonly record struct. * Fix "GameHelpers-> ApplySize" return value override actual values. * Refactor: Fix tests using "with" for "readonly record struct" initializations. * Clean up * Only use the "with" keyword when it is needed (avoid unnecessary copy). * Remove old struct custom tests.
KinsonDigital · Nov 27, 2023 · 1690172 · 1690172
1 parent 76bdcf9
commit 1690172
Show file tree

Hide file tree

Showing 9 changed files with 114 additions and 564 deletions.
diff --git a/Testing/VelaptorTests/GameHelpersTests.cs b/Testing/VelaptorTests/GameHelpersTests.cs
@@ -338,21 +338,23 @@ public void ApplySize_WithRectangleF_ReturnsCorrectResult()
     public void ApplySize_WithGlyphMetrics_ReturnsCorrectResult()
     {
         // Arrange
-        var metrics = default(GlyphMetrics);
-        metrics.Glyph = 'V';
-        metrics.GlyphBounds = new RectangleF(2f, 4f, 6f, 8f);
-        metrics.Ascender = 2f;
-        metrics.Descender = 2f;
-        metrics.HorizontalAdvance = 2f;
-        metrics.HoriBearingX = 2f;
-        metrics.HoriBearingY = 2f;
-        metrics.GlyphWidth = 2f;
-        metrics.GlyphHeight = 2f;
-        metrics.XMin = 2f;
-        metrics.XMax = 2f;
-        metrics.YMin = 2f;
-        metrics.YMax = 2f;
-        metrics.CharIndex = 123u;
+        var metrics = new GlyphMetrics
+        {
+            Glyph = 'V',
+            GlyphBounds = new RectangleF(2f, 4f, 6f, 8f),
+            Ascender = 2f,
+            Descender = 2f,
+            HorizontalAdvance = 2f,
+            HoriBearingX = 2f,
+            HoriBearingY = 2f,
+            GlyphWidth = 2f,
+            GlyphHeight = 2f,
+            XMin = 2f,
+            XMax = 2f,
+            YMin = 2f,
+            YMax = 2f,
+            CharIndex = 123u,
+        };
 
         // Act
         var actual = metrics.ApplySize(2f);