Skip to content

Handle hoisted Catch local in state machine rewriter - #883

Merged
cston merged 12 commits into
dotnet:masterfrom
cston:1115030
Mar 2, 2015
Merged

Handle hoisted Catch local in state machine rewriter#883
cston merged 12 commits into
dotnet:masterfrom
cston:1115030

Conversation

@cston

@cston cston commented Feb 26, 2015

Copy link
Copy Markdown
Contributor

Fixes #863

@cston

cston commented Feb 26, 2015

Copy link
Copy Markdown
Contributor Author

@gafter, @AlekseyTs

@gafter

gafter commented Feb 26, 2015

Copy link
Copy Markdown
Member

Is there some bug that this is fixing?

@cston

cston commented Feb 26, 2015

Copy link
Copy Markdown
Contributor Author

Fixes #863

@gafter

gafter commented Feb 27, 2015

Copy link
Copy Markdown
Member

👍

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The initializer 'Nothing' is not used on any control path and looks unnecessary.

cston added a commit that referenced this pull request Mar 2, 2015
Handle hoisted Catch local in state machine rewriter

Fixes #863
@cston
cston merged commit 6e5efac into dotnet:master Mar 2, 2015
@cston
cston deleted the 1115030 branch March 2, 2015 17:39
dibarbet pushed a commit that referenced this pull request Aug 18, 2026
Add ReferenceAssemblies.Net.Net60
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

VB assert failure: "captured local should not need rewriting"

5 participants