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

Remove unused local #40967

Merged
merged 1 commit into from
Aug 23, 2020
Merged

Remove unused local #40967

merged 1 commit into from
Aug 23, 2020

Conversation

seonyounggg
Copy link
Contributor

This fixes a part of #30457
remove unused local variable dwErrorCode at line 248.
Please check it

issue #30457
remove unused local variable dwErrorCode at line 248
@ghost
Copy link

ghost commented Aug 18, 2020

Tagging subscribers to this area: @bartonjs, @vcsjones, @krwq
See info in area-owners.md if you want to be subscribed.

@jkotas jkotas removed the area-Meta label Aug 18, 2020
@seonyounggg
Copy link
Contributor Author

@jkotas I think It's okay to remove that line but two checks failed. Can you check this please?

Copy link
Member

@jkotas jkotas left a comment

Choose a reason for hiding this comment

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

Thanks

@jkotas jkotas merged commit 4b431b5 into dotnet:master Aug 23, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 7, 2020
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