Allocate exactly the space needed when creating a sourcetext #71886
Conversation
| { | ||
| // char pooled memory : 8K * 256 = 2MB | ||
| private const int CharBufferSize = 4 * 1024; | ||
| private static readonly ObjectPool<char[]> CharArrayPool = new(() => new char[CharBufferSize], 256); |
There was a problem hiding this comment.
moved these here as this is the only place it is used. and it is easier to validate proper usage.
src/Workspaces/Core/Portable/Shared/Extensions/SourceTextExtensions.cs
Outdated
Show resolved
Hide resolved
| writer.WriteInt32(numberOfChunks); | ||
|
|
||
| // write whole chunks | ||
| try |
There was a problem hiding this comment.
view with whitespace off.
| var elementType = typeof(byte); | ||
| Debug.Assert(s_typeMap[elementType] == TypeCode.UInt8); | ||
| var elementType = typeof(T); | ||
| Contract.ThrowIfFalse(s_typeMap.ContainsKey(elementType)); |
There was a problem hiding this comment.
i don't mind this too much. so we get a nice contract failure if we defy expectations here.
There was a problem hiding this comment.
Not a big deal, but not sure why this is less clear:
Contract.ThrowIfFalse(s_typeMap.TryGetValue(elementType, out var actualType));
Contract.ThrowIfTrue(expectedType != actualType);
There was a problem hiding this comment.
ah, misunderstood. yes, that's nicer. will do in followup.
| } | ||
|
|
||
| /// <summary> | ||
| /// Write an array of bytes. The array data is provided as a <see |
There was a problem hiding this comment.
can fix in followup :)
| Action<BinaryWriter, T[], int> write) | ||
| { | ||
| var spanLength = span.Length; | ||
| var buffer = System.Buffers.ArrayPool<T>.Shared.Rent(Math.Min(spanLength, rentLength)); |
There was a problem hiding this comment.
offline: for straight IO buffers, this feels more in line with what the runtime does.
| /// </summary> | ||
| /// <param name="span">The array data.</param> | ||
| public void WriteValue(ReadOnlySpan<byte> span) | ||
| public void WriteSpan(ReadOnlySpan<byte> span) |
No description provided.