Skip to content
This repository was archived by the owner on Nov 1, 2020. It is now read-only.

Add Lazy<T>(T value) constructor#1538

Merged
jkotas merged 1 commit into
masterfrom
unknown repository
Jul 18, 2016
Merged

Add Lazy<T>(T value) constructor#1538
jkotas merged 1 commit into
masterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jul 17, 2016

Corresponds to dotnet/coreclr#6304.

@dnfclas
Copy link
Copy Markdown

dnfclas commented Jul 17, 2016

Hi @jasonwilliams200OK, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.

TTYL, DNFBOT;

// concurrently.
/// </remarks>
public Lazy(T value)
{
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This doesn't compile. The field is named _boxed here.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Ah done thanks. IMO all this shared code should be shared by having a submodule to something like core-commons repo.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants