Skip to content

Conversation

@zhenlei520
Copy link
Contributor

No description provided.

@zhenlei520 zhenlei520 requested a review from doddgu May 6, 2022 09:22

namespace Masa.Contrib.Data.Mapping.Mapster.Internal;

internal static class ConcurrentDictionaryExtensions
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think you need to copy and paste this lazy method all over the place, it's going to be used a lot.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Adjust the GetOrAdd method provided for Memory

Copy link
Contributor

@doddgu doddgu left a comment

Choose a reason for hiding this comment

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

LGTM

@doddgu doddgu merged commit 4a2f393 into main May 7, 2022
@doddgu doddgu deleted the feature/mapping branch May 7, 2022 01:19
@zhenlei520 zhenlei520 mentioned this pull request May 17, 2022
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants