Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

CollectionsMarshal.AsSpan allow null refs #26903

Merged
merged 1 commit into from
Sep 26, 2019

Conversation

benaadams
Copy link
Member

@benaadams benaadams commented Sep 26, 2019

As per #26867 (comment)

@jkotas @GrabYourPitchforks @ahsonkhan @tannergooding does this change require api review?

@tannergooding tannergooding merged commit d0ad047 into dotnet:master Sep 26, 2019
@ahsonkhan
Copy link
Member

Lol, race condition. Usually "approves" post-merge don't show up:
image

@ahsonkhan
Copy link
Member

ahsonkhan commented Sep 26, 2019

does this change require api review?

We can quickly revisit this in the next review and also close on us not adding AsReadOnlySpan (https://github.com/dotnet/corefx/issues/31597#issuecomment-534819414). It doesn't need to block any PRs/merges though.

Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/corefx that referenced this pull request Sep 26, 2019
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/mono that referenced this pull request Sep 26, 2019
marek-safar pushed a commit to mono/mono that referenced this pull request Sep 26, 2019
ViktorHofer pushed a commit to dotnet/corefx that referenced this pull request Sep 26, 2019
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/corert that referenced this pull request Sep 26, 2019
jkotas pushed a commit to dotnet/corert that referenced this pull request Sep 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants