Skip to content

Conversation

@adamreeve
Copy link
Contributor

Small tidy up following on from #600 where I noticed there were a few places where we create an empty array with Array.Empty<TPhysical>() just to pass to a method that expects a ReadOnlySpan. To skip the conversion from an array to span and make it clearer what types are being used I've changed these to create an empty ReadOnlySpan directly. I also changed some places that pass null to use ReadOnlySpan.Empty or Span.Empty too for consistency.

@adamreeve adamreeve merged commit 250ff6b into G-Research:master Dec 12, 2025
60 of 61 checks passed
@adamreeve adamreeve deleted the use_empty_span branch December 12, 2025 08:39
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