Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Remove some unnecessary List-related allocation from EventSource#44026

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

Merged
stephentoub merged 1 commit intodotnet:masterfromstephentoub:eventsourcealloc
Oct 29, 2020

Conversation

@stephentoub
Copy link
Member

Fixes#43989

am11 and theolivenbaum reacted with thumbs up emojitheolivenbaum reacted with hooray emoji
@ghost
Copy link

Tagging subscribers to this area:@tarekgh,@tommcdon,@pjanotti
See info in area-owners.md if you want to be subscribed.

@tarekgh
Copy link
Member

CC@noahfalk

@GrabYourPitchforks
Copy link
Member

@stephentoub Did you intend for this PR to resolve#43989? That issue seemingly tracks a new feature request.

@stephentoub
Copy link
MemberAuthor

Did you intend for this PR to resolve#43989? That issue seemingly tracks a new feature request.

I don't see any new feature request in the comments from the dev who opened the issue?
#43989 (comment)

What feature?

@GrabYourPitchforks
Copy link
Member

Ah, I misinterpreted that issue as a "please allow me to provide my own backing array toList<T>." Carry on! :)

@stephentoub
Copy link
MemberAuthor

😄

@stephentoubstephentoub merged commit0283591 intodotnet:masterOct 29, 2020
@stephentoubstephentoub deleted the eventsourcealloc branchOctober 29, 2020 23:54
@ghostghost locked asresolvedand limited conversation to collaboratorsDec 6, 2020
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@tarekghtarekghtarekgh approved these changes

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Reduce List<object> allocations by passing the initial array

4 participants

@stephentoub@tarekgh@GrabYourPitchforks@Dotnet-GitSync-Bot
[8]ページ先頭
©2009-2025 Movatter.jp