fix: Fixes ValueStringBuilder initial rented buffer (#1111)

Fixes issue with ValueStringBuilder having an empty initial buffer.
This commit is contained in:
Kamron Batman 2022-07-03 08:14:54 -07:00 committed by GitHub
parent c6c644c966
commit 8303c74379
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 4 deletions

View file

@ -51,10 +51,9 @@ public ref struct ValueStringBuilder
public ValueStringBuilder(int initialCapacity, bool mt = false)
{
_mt = mt;
_arrayToReturnToPool = null;
_chars = _arrayToReturnToPool;
_length = 0;
_arrayToReturnToPool = ArrayPool.Rent(initialCapacity);
_arrayToReturnToPool = (_mt ? ArrayPool<char>.Shared : STArrayPool<char>.Shared).Rent(initialCapacity);
_chars = _arrayToReturnToPool;
}
public int Length => _length;

View file

@ -96,7 +96,7 @@ namespace Server.Gumps
}
case 3: // Apply
{
using var propsBuilder = new ValueStringBuilder(64);
using var propsBuilder = ValueStringBuilder.Create();
bool first = true;
foreach (var attr in MobileAttributes)
{