Use other JIT hints

Author: linkdotnet

src/LinkDotNet.StringBuilder/ValueStringBuilder.AppendJoin.cs

@@ -42,7 +42,7 @@ public void AppendJoin<T>(ReadOnlySpan<char> separator, IEnumerable<T> values)
     public void AppendJoin<T>(char separator, IEnumerable<T> values)
         => AppendJoinInternalChar(separator, values);
 
-    [MethodImpl(MethodImplOptions.AggressiveInlining)]
+    [MethodImpl(MethodImplOptions.AggressiveOptimization)]
     private void AppendJoinInternalString<T>(ReadOnlySpan<char> separator, IEnumerable<T> values)
     {
         ArgumentNullException.ThrowIfNull(values);
@@ -68,7 +68,7 @@ private void AppendJoinInternalString<T>(ReadOnlySpan<char> separator, IEnumerab
         }
     }
 
-    [MethodImpl(MethodImplOptions.AggressiveInlining)]
+    [MethodImpl(MethodImplOptions.AggressiveOptimization)]
     private void AppendJoinInternalChar<T>(char separator, IEnumerable<T> values)
     {
         ArgumentNullException.ThrowIfNull(values);

src/LinkDotNet.StringBuilder/ValueStringBuilder.cs

@@ -114,6 +114,7 @@ public ValueStringBuilder(ReadOnlySpan<char> initialText)
     /// </summary>
     /// <param name="range">The range that will be retrieved.</param>
     /// <returns>The <see cref="string"/> instance.</returns>
+    [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public readonly string ToString(Range range)
     {
         var (offset, length) = range.GetOffsetAndLength(bufferPosition);
@@ -296,9 +297,10 @@ public readonly void Dispose()
     /// <summary>
     /// Reverses the sequence of elements in this instance.
     /// </summary>
+    [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public readonly void Reverse() => buffer[..bufferPosition].Reverse();
 
-    [MethodImpl(MethodImplOptions.AggressiveInlining)]
+    [MethodImpl(MethodImplOptions.AggressiveOptimization)]
     private void Grow(int capacity = 0)
     {
         var size = buffer.Length == 0 ? 8 : buffer.Length;

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

@@ -168,4 +168,15 @@ public void GivenMemorySlice_ShouldAppend()
 
         builder.ToString().Should().Be("ccccc");
     }
+
+    [Fact]
+    public void GivenAStringWithWhitespace_WhenTrimIsCalled_ThenTheStringShouldBeTrimmed()
+    {
+        using var builder = new ValueStringBuilder();
+        builder.Append("  Hello World  ");
+
+        builder.Trim();
+
+        builder.ToString().Should().Be("Hello World");
+    }
 }