It looks like memory leak in Utf16ValueStringBuilder.Insert

Original buffer is not released before assigning new buffer

https://github.com/Cysharp/ZString/blob/98379750ad4aae4c75b2ea1559d47b6931d866dc/src/ZString/Utf16ValueStringBuilder.cs#L309-L314

For example `Grow` releases buffer before assigning new one

https://github.com/Cysharp/ZString/blob/98379750ad4aae4c75b2ea1559d47b6931d866dc/src/ZString/Utf16ValueStringBuilder.cs#L139-L146

	int remainLnegth = this.index - index;
	buffer.AsSpan(index, remainLnegth).CopyTo(newBuffer.AsSpan(newBufferIndex));

	buffer = newBuffer;
	this.index = newBufferIndex + remainLnegth;
	}

	buffer.CopyTo(newBuffer, 0);
	if (buffer.Length != ThreadStaticBufferSize)
	{
	ArrayPool<char>.Shared.Return(buffer);
	}

	buffer = newBuffer;
	}

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

It looks like memory leak in Utf16ValueStringBuilder.Insert #36

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

It looks like memory leak in Utf16ValueStringBuilder.Insert #36

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions