Skip to content

[Proposal] Adds TextSpan.AsReadOnlySpan()#169

Merged
nblumhardt merged 3 commits intodatalust:devfrom
aka-nse:feature/as-readonly-span
Sep 7, 2025
Merged

[Proposal] Adds TextSpan.AsReadOnlySpan()#169
nblumhardt merged 3 commits intodatalust:devfrom
aka-nse:feature/as-readonly-span

Conversation

@aka-nse
Copy link
Copy Markdown
Contributor

@aka-nse aka-nse commented Sep 4, 2025

This proposal is a part of #168, with a new unit test code.


This pull request introduces a feature:

  • TextSpan.AsReadOnlySpan():
    provides the foundation for using modern span-based APIs, paving the way for improved performance.

Impact

  • External package dependency: New reference to System.Memory is introduced only for '$(TargetFramework)'=='netstandard2.0'.

<None Include="..\..\asset\Superpower-White-400px.png" Pack="true" Visible="false" PackagePath="" />
</ItemGroup>
<ItemGroup>
<PackageReference Include="System.Memory" Version="4.6.3" />
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we please also keep the dependency conditional on netstandard2.0, as in the earlier PR? Thanks!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nblumhardt
Sorry, it seems a revision was missed.
I fixed it.

@nblumhardt nblumhardt merged commit 9980912 into datalust:dev Sep 7, 2025
1 check passed
@nblumhardt
Copy link
Copy Markdown
Member

Thanks!

@nblumhardt nblumhardt mentioned this pull request Apr 27, 2026
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