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

map TcpListener APM methods to Task-based methods #40476

Merged
merged 1 commit into from
Aug 6, 2020

Conversation

geoffkizer
Copy link
Contributor

This removes the custom implementation of APM methods on TcpListener by mapping them to Task-based methods and removes the dependency on LazyAsyncResult in this code.

@Dotnet-GitSync-Bot
Copy link
Collaborator

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@ghost
Copy link

ghost commented Aug 6, 2020

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

@geoffkizer geoffkizer merged commit 2021d54 into dotnet:master Aug 6, 2020
Jacksondr5 pushed a commit to Jacksondr5/runtime that referenced this pull request Aug 10, 2020
Co-authored-by: Geoffrey Kizer <geoffrek@windows.microsoft.com>
@karelz karelz added this to the 5.0.0 milestone Aug 18, 2020
antonfirsov added a commit that referenced this pull request Sep 7, 2020
According to the old (pre- #40476) code and the documentation, we should throw ObjectDisposedException in EndAcceptSocket and EndAcceptTcpClient.

Fixes #41585
@geoffkizer geoffkizer deleted the tcplistenerasync branch November 7, 2020 23:41
@ghost ghost locked as resolved and limited conversation to collaborators Dec 8, 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.

None yet

5 participants