- Notifications
You must be signed in to change notification settings - Fork5.2k
Add MemoryMarshal.CreateReadOnlySpanFromNullTerminated#47539
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
ghost commentedJan 27, 2021
Note regarding the This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change. |
src/libraries/System.Memory/tests/MemoryMarshal/CreateReadOnlySpanFromNullTerminated.cs OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
cbc90a1 to4888b01Comparesrc/libraries/System.Memory/tests/MemoryMarshal/CreateReadOnlySpanFromNullTerminated.cs OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
We'd want to port these changes to the actual docs site.
https://docs.microsoft.com/en-us/dotnet/api/system.runtime.interopservices.memorymarshal.createspan?view=net-5.0
src/libraries/System.Private.CoreLib/src/System/Runtime/InteropServices/MemoryMarshal.cs OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
src/libraries/System.Private.CoreLib/src/System/Runtime/InteropServices/MemoryMarshal.cs OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
src/libraries/System.Private.CoreLib/src/System/Runtime/InteropServices/MemoryMarshal.cs OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
stephentoub commentedJan 29, 2021
/azp list |
stephentoub commentedJan 29, 2021
/azp run runtime-libraries-coreclr outerloop |
| Azure Pipelines successfully started running 1 pipeline(s). |
Fixes#40202
cc:@jkotas,@GrabYourPitchforks