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

Add Lazy<T>(T value) ctor to ref#10129

Closed
ghost wants to merge 1 commit into
masterfrom
unknown repository
Closed

Add Lazy<T>(T value) ctor to ref#10129
ghost wants to merge 1 commit into
masterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jul 18, 2016

Also added a theory test.

Depends upon dotnet/coreclr#6304.

Related to #10127 and #2578.

Also added a theory test.
@stephentoub stephentoub added the * NO MERGE * The PR is not ready for merge yet (see discussion for detailed reasons) label Jul 18, 2016
@stephentoub
Copy link
Copy Markdown
Member

This will prevent the contract from being used with all existing versions of desktop. I'm not sure we're prepared to do that yet.
cc: @weshaggard, @terrajobst

@weshaggard
Copy link
Copy Markdown
Member

This will prevent the contract from being used with all existing versions of desktop. I'm not sure we're prepared to do that yet.
cc: @weshaggard, @terrajobst

Agreed. We cannot put this API into System.Runtime yet, for the next version of System.Runtime we plan to expose a bunch more APIs that already exist on the desktop framework so we need to be sure we don't have any new APIs that aren't there yet. Once we get to a stable point there we can add APIs that don't exist on desktop yet.

@stephentoub
Copy link
Copy Markdown
Member

Thanks, @jasonwilliams200OK. Per the previous comments, I'm going to close this out for now. If there isn't an issue currently tracking this, please feel free to open one, though it sounds like there already is.

@karelz karelz modified the milestone: 1.1.0 Dec 3, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

* NO MERGE * The PR is not ready for merge yet (see discussion for detailed reasons)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants