Skip to content

Avoid closure for GetOrAdd - #62317

Merged
danmoseley merged 1 commit into
dotnet:mainfrom
meziantou:closure
Dec 3, 2021
Merged

Avoid closure for GetOrAdd#62317
danmoseley merged 1 commit into
dotnet:mainfrom
meziantou:closure

Conversation

@meziantou

Copy link
Copy Markdown
Contributor

Description

Use the parameter of the lambda instead of creating a closure

@ghostghost added community-contribution Indicates that the PR has been added by a community member area-System.Data labels Dec 3, 2021
@ghost

ghost commented Dec 3, 2021

Copy link
Copy Markdown

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

Issue Details

Description

Use the parameter of the lambda instead of creating a closure

Author:meziantou
Assignees:-
Labels:

area-System.Data, community-contribution

Milestone:-

@stephentoubstephentoub left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Thanks.

@meziantou
meziantou marked this pull request as ready for review December 3, 2021 04:10
@danmoseley
danmoseley merged commit 42224c6 into dotnet:mainDec 3, 2021
@ghostghost locked as resolved and limited conversation to collaborators Jan 2, 2022
@meziantou
meziantou deleted the closure branch January 28, 2022 22:44
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Datacommunity-contributionIndicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@meziantou@stephentoub@danmoseley