Skip to content

Rename NativeMemory.ZeroMemory - #69948

Merged
jkotas merged 1 commit into
dotnet:mainfrom
jkotas:ZeroMemory
May 29, 2022
Merged

jkotas merged 1 commit into
dotnet:mainfrom
jkotas:ZeroMemory

Conversation

@jkotas

@jkotas jkotas commented May 28, 2022

Copy link
Copy Markdown
Member

- Rename NativeMemory.ZeroMemory to NativeMemory.Clear. Contributes to dotnet#69606
- Delete internal Buffer.ZeroMemory and replace it with NativeMemory.Clear
@ghost

ghost commented May 28, 2022

Copy link
Copy Markdown

Note regarding the new-api-needs-documentation label:

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.

@ghost

ghost commented May 28, 2022

Copy link
Copy Markdown

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

Issue Details
Author: jkotas
Assignees: jkotas
Labels:

area-System.Memory, new-api-needs-documentation

Milestone: -

@jkotas

jkotas commented May 29, 2022

Copy link
Copy Markdown
Member Author

The test failure is #69474

@jkotas
jkotas merged commit 7aa4010 into dotnet:main May 29, 2022
@jkotas
jkotas deleted the ZeroMemory branch May 29, 2022 01:52
@adamsitnik adamsitnik added this to the 7.0.0 milestone Jun 23, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Jul 23, 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.

3 participants