refactor: rename PadLeft and PadRight methods to AppendPadLeft and AppendPadRight

Author: linkdotnet

CHANGELOG.md

@@ -6,6 +6,9 @@ All notable changes to **ValueStringBuilder** will be documented in this file. T
 
 ## [Unreleased]
 
+### Changed
+- Renamed `PadLeft(ReadOnlySpan<char>, int, char)` and `PadRight(ReadOnlySpan<char>, int, char)` to `AppendPadLeft(ReadOnlySpan<char>, int, char)` and `AppendPadRight(ReadOnlySpan<char>, int, char)` respectively.
+
 ## [3.1.0] - 2025-10-31
 
 ### Added

src/LinkDotNet.StringBuilder/ValueStringBuilder.Pad.cs

@@ -48,19 +48,19 @@ public void PadRight(int totalWidth, char paddingChar)
     /// Appends the source string padded on the left with the given character to reach the specified total width.
     /// </summary>
     /// <param name="source">The source string to pad and append.</param>
-    /// <param name="totalWidth">Total width of the padded string.</param>
+    /// <param name="sourceTotalWidth">Total width of the padded string.</param>
     /// <param name="paddingChar">Character to pad the string with. Defaults to space.</param>
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
-    public void PadLeft(ReadOnlySpan<char> source, int totalWidth, char paddingChar = ' ')
+    public void AppendPadLeft(scoped ReadOnlySpan<char> source, int sourceTotalWidth, char paddingChar = ' ')
     {
-        if (totalWidth <= source.Length)
+        if (sourceTotalWidth <= source.Length)
         {
             Append(source);
             return;
         }
 
-        var padding = totalWidth - source.Length;
-        EnsureCapacity(bufferPosition + totalWidth);
+        var padding = sourceTotalWidth - source.Length;
+        EnsureCapacity(bufferPosition + sourceTotalWidth);
 
         buffer.Slice(bufferPosition, padding).Fill(paddingChar);
         bufferPosition += padding;
@@ -73,19 +73,19 @@ public void PadLeft(ReadOnlySpan<char> source, int totalWidth, char paddingChar
     /// Appends the source string padded on the right with the given character to reach the specified total width.
     /// </summary>
     /// <param name="source">The source string to pad and append.</param>
-    /// <param name="totalWidth">Total width of the padded string.</param>
+    /// <param name="sourceTotalWidth">Total width of the padded string.</param>
     /// <param name="paddingChar">Character to pad the string with. Defaults to space.</param>
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
-    public void PadRight(ReadOnlySpan<char> source, int totalWidth, char paddingChar = ' ')
+    public void AppendPadRight(scoped ReadOnlySpan<char> source, int sourceTotalWidth, char paddingChar = ' ')
     {
-        if (totalWidth <= source.Length)
+        if (sourceTotalWidth <= source.Length)
         {
             Append(source);
             return;
         }
 
-        var padding = totalWidth - source.Length;
-        EnsureCapacity(bufferPosition + totalWidth);
+        var padding = sourceTotalWidth - source.Length;
+        EnsureCapacity(bufferPosition + sourceTotalWidth);
 
         source.CopyTo(buffer[bufferPosition..]);
         bufferPosition += source.Length;

tests/LinkDotNet.StringBuilder.UnitTests/ValueStringBuilder.Pad.Tests.cs

@@ -47,7 +47,7 @@ public void ShouldPadLeftWithSource()
     {
         using var stringBuilder = new ValueStringBuilder();
 
-        stringBuilder.PadLeft("Hello".AsSpan(), 10, ' ');
+        stringBuilder.AppendPadLeft("Hello".AsSpan(), 10, ' ');
 
         stringBuilder.ToString().ShouldBe("     Hello");
     }
@@ -57,7 +57,7 @@ public void ShouldPadRightWithSource()
     {
         using var stringBuilder = new ValueStringBuilder();
 
-        stringBuilder.PadRight("Hello".AsSpan(), 10, ' ');
+        stringBuilder.AppendPadRight("Hello".AsSpan(), 10, ' ');
 
         stringBuilder.ToString().ShouldBe("Hello     ");
     }
@@ -67,7 +67,7 @@ public void ShouldPadLeftWithSourceAndCustomChar()
     {
         using var stringBuilder = new ValueStringBuilder();
 
-        stringBuilder.PadLeft("42".AsSpan(), 5, '0');
+        stringBuilder.AppendPadLeft("42".AsSpan(), 5, '0');
 
         stringBuilder.ToString().ShouldBe("00042");
     }
@@ -77,7 +77,7 @@ public void ShouldPadRightWithSourceAndCustomChar()
     {
         using var stringBuilder = new ValueStringBuilder();
 
-        stringBuilder.PadRight("Test".AsSpan(), 8, '*');
+        stringBuilder.AppendPadRight("Test".AsSpan(), 8, '*');
 
         stringBuilder.ToString().ShouldBe("Test****");
     }
@@ -87,7 +87,7 @@ public void GivenTotalWidthIsSmallerThanSourceLength_WhenPadLeftWithSource_ThenS
     {
         using var stringBuilder = new ValueStringBuilder();
 
-        stringBuilder.PadLeft("Hello".AsSpan(), 3, ' ');
+        stringBuilder.AppendPadLeft("Hello".AsSpan(), 3, ' ');
 
         stringBuilder.ToString().ShouldBe("Hello");
     }
@@ -97,7 +97,7 @@ public void GivenTotalWidthIsSmallerThanSourceLength_WhenPadRightWithSource_Then
     {
         using var stringBuilder = new ValueStringBuilder();
 
-        stringBuilder.PadRight("Hello".AsSpan(), 3, ' ');
+        stringBuilder.AppendPadRight("Hello".AsSpan(), 3, ' ');
 
         stringBuilder.ToString().ShouldBe("Hello");
     }
@@ -107,8 +107,8 @@ public void ShouldPadLeftWithSourceMultipleTimes()
     {
         using var stringBuilder = new ValueStringBuilder();
 
-        stringBuilder.PadLeft("A".AsSpan(), 3, '-');
-        stringBuilder.PadLeft("B".AsSpan(), 3, '-');
+        stringBuilder.AppendPadLeft("A".AsSpan(), 3, '-');
+        stringBuilder.AppendPadLeft("B".AsSpan(), 3, '-');
 
         stringBuilder.ToString().ShouldBe("--A--B");
     }
@@ -118,8 +118,8 @@ public void ShouldPadRightWithSourceMultipleTimes()
     {
         using var stringBuilder = new ValueStringBuilder();
 
-        stringBuilder.PadRight("A".AsSpan(), 3, '-');
-        stringBuilder.PadRight("B".AsSpan(), 3, '-');
+        stringBuilder.AppendPadRight("A".AsSpan(), 3, '-');
+        stringBuilder.AppendPadRight("B".AsSpan(), 3, '-');
 
         stringBuilder.ToString().ShouldBe("A--B--");
     }
@@ -129,7 +129,7 @@ public void ShouldPadLeftWithSourceAndExistingContent()
     {
         using var stringBuilder = new ValueStringBuilder("Start ");
 
-        stringBuilder.PadLeft("End".AsSpan(), 10, '.');
+        stringBuilder.AppendPadLeft("End".AsSpan(), 10, '.');
 
         stringBuilder.ToString().ShouldBe("Start .......End");
     }
@@ -139,7 +139,7 @@ public void ShouldPadRightWithSourceAndExistingContent()
     {
         using var stringBuilder = new ValueStringBuilder("Start ");
 
-        stringBuilder.PadRight("End".AsSpan(), 10, '.');
+        stringBuilder.AppendPadRight("End".AsSpan(), 10, '.');
 
         stringBuilder.ToString().ShouldBe("Start End.......");
     }