fix: Grow allowed truncating of string

Author: linkdotnet

CHANGELOG.md

@@ -9,6 +9,9 @@ All notable changes to **ValueStringBuilder** will be documented in this file. T
 ### Added
 - Added the `scoped` keyword to simplify code and allow more scenarios for the user
 
+### Fixed
+- `Grow` allowed values, which would truncate the internally represented string
+
 ## [1.3.0] - 2022-07-25
 
 ### Fixed

src/LinkDotNet.StringBuilder/ValueStringBuilder.cs

@@ -259,9 +259,7 @@ private void Grow(int capacity = 0)
     {
         var currentSize = buffer.Length;
 
-        // This could lead to the potential problem that an user sets the capacity smaller than the current length
-        // which would lead to a truncated string.
-        var newSize = capacity > 0 ? capacity : currentSize * 2;
+        var newSize = capacity > currentSize ? capacity : currentSize * 2;
         var rented = ArrayPool<char>.Shared.Rent(newSize);
         buffer.CopyTo(rented);
         var toReturn = arrayFromPool;