Skip to content

Conversation

halgab
Copy link
Contributor

@halgab halgab commented Nov 10, 2023

Summary

I've spotted what looks like a bad copy/paste from the corresponding string extension method for method MemoryExtensions.AsSpan<T>(this T[]? array, int start, int length)

@halgab halgab requested a review from a team as a code owner November 10, 2023 13:56
@ghost ghost added community-contribution Indicates that the PR has been added by a community member area-System.Runtime labels Nov 10, 2023
Copy link

Learn Build status updates of commit 4c4c73f:

✅ Validation status: passed

File Status Preview URL Details
xml/System/MemoryExtensions.xml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@halgab halgab changed the title Fix docs of method MemoryExtensions.AsSpan<T>(this T[]? array, int st… Fix docs of MemoryExtensions.AsSpan<T>(this T[]? array, int start, int length) Nov 10, 2023
Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

Thanks @halgab

I've reviewed this, and it LGTM. I'll :shipit: now.

@BillWagner BillWagner merged commit 0c2ed44 into dotnet:main Nov 10, 2023
@halgab halgab deleted the patch-1 branch November 10, 2023 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-System.Runtime community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants