Skip to content
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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Rename MemoryHandle.PinnedPointer to Pointer #23734

Closed
KrzysztofCwalina opened this issue Oct 4, 2017 · 3 comments
Closed

Rename MemoryHandle.PinnedPointer to Pointer #23734

KrzysztofCwalina opened this issue Oct 4, 2017 · 3 comments
Assignees
@ahsonkhan
Labels
api-approved API was approved in API review, it can be implemented area-System.Memory
Milestone
2.1.0

Comments

@KrzysztofCwalina
Copy link
Member

No description provided.

@karelz
Copy link
Member

karelz commented Oct 17, 2017

Discussed originally here: https://github.com/dotnet/corefx/issues/23974#issuecomment-333943725

@terrajobst
Copy link
Member

terrajobst commented Oct 17, 2017 •
edited

Video

Looks good as proposed. Also, rename the constructor.

AND NEXT TIME INCLUDE THE API REF.

@karelz
Copy link
Member

karelz commented Oct 17, 2017

FYI: The API review discussion was recorded - see https://youtu.be/BEBq3__WfDc?t=856 (4 min duration)

@msftgits msftgits transferred this issue from dotnet/corefx Jan 31, 2020
@msftgits msftgits added this to the 2.1.0 milestone Jan 31, 2020
@dotnet dotnet locked as resolved and limited conversation to collaborators Dec 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ahsonkhan ahsonkhan

Labels
api-approved API was approved in API review, it can be implemented area-System.Memory
Projects
None yet
Milestone
2.1.0
Development

No branches or pull requests
5 participants
@terrajobst @karelz @ahsonkhan @msftgits @KrzysztofCwalina