Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DRAFT prototype for 16167 #16268

Closed
wants to merge 1 commit into from
Closed

DRAFT prototype for 16167 #16268

wants to merge 1 commit into from

Conversation

gafter
Copy link
Member

@gafter gafter commented Jan 5, 2017

Possible approach for fixing #16167

@AlekseyTs I'd like your opinion about this approach for addressing the bug underlying #16195, which is that we currently do not give any scope to variables declared in default parameter values of local function declarations. If you think this is a reasonable approach, I'll complete the work based on this direction. If not, what would you recommend?

Possible approach for fixing dotnet#16195
@gafter gafter added Area-Compilers PR For Personal Review Only The PR doesn’t require anyone other than the developer to review it. labels Jan 5, 2017
@gafter gafter added this to the 2.0 (RC.3) milestone Jan 5, 2017
@gafter gafter changed the title DRAFT prototype for 16195 DRAFT prototype for 16167 Jan 5, 2017
@gafter gafter closed this Jan 6, 2017
@gafter gafter deleted the master-16195 branch April 18, 2017 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Compilers cla-already-signed PR For Personal Review Only The PR doesn’t require anyone other than the developer to review it.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants