Skip to content

Conversation

@TimothyMakkison
Copy link
Contributor

@TimothyMakkison TimothyMakkison commented Dec 23, 2025

Replaces List<string> with Span<string?>, modern collection expressions will generate a fixed length struct, null initialise it and then cast into Span<string?>.

This trick could probably be used in other areas, along with ValueListBuilder and ValueStringBuilder.

Not sure if it's worth using Debug.Assert to ensure that elements contains maxElements + 1 values, I don't see TUnit using it very much.

@thomhurst thomhurst merged commit 18a2226 into thomhurst:main Dec 23, 2025
8 of 10 checks passed
@claude claude bot mentioned this pull request Dec 24, 2025
1 task
This was referenced Dec 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants