Skip to content

Conversation

@elinor-fung
Copy link
Member

Convert some existing LibraryImports to use ReadOnlySpan/Span directly. There's definitely more that we could switch over, but this should ensure we have coverage/usage in the runtime on different platforms.

@ghost
Copy link

ghost commented Aug 2, 2022

Tagging subscribers to this area: @dotnet/interop-contrib
See info in area-owners.md if you want to be subscribed.

Issue Details

Convert some existing LibraryImports to use ReadOnlySpan/Span directly. There's definitely more that we could switch over, but this should ensure we have coverage/usage in the runtime on different platforms.

Author: elinor-fung
Assignees: -
Labels:

area-System.Runtime.InteropServices

Milestone: -

Copy link
Member

@jkoritzinsky jkoritzinsky left a comment

Choose a reason for hiding this comment

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

🎉🎉

Copy link
Member

@AaronRobinsonMSFT AaronRobinsonMSFT left a comment

Choose a reason for hiding this comment

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

:shipit:

@elinor-fung elinor-fung merged commit 8e4092e into dotnet:main Aug 3, 2022
@elinor-fung elinor-fung deleted the useSpanMarshaller branch August 3, 2022 00:40
@ghost ghost locked as resolved and limited conversation to collaborators Sep 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants