Replacement of APM usage (Begin/EndGetResponse, etc.) with TAP (GetResponceAsync) and async/await #4

Merged
merged 3 commits into from Aug 22, 2013

Conversation

Projects
None yet
2 participants
@dlhartveld
Contributor

dlhartveld commented Aug 9, 2013

As part of a research project at the University of Illinois at Urbana-Champaign, we're developing a refactoring tool that replaces instances of the APM pattern with corresponding TAP method calls and the async/await keywords. I've applied it to this project and found one call to an APM method.

This pull request replaces those existing calls to APM Begin/End* methods with functionally equivalent TAP constructs and the async/await keywords.

This commit contains the actual code-level changes: fb853bf

I also added a dependency that is needed: Microsoft.Bcl.Async. This package improves support for async/await-based programming for older frameworks, or frameworks that are missing functionality (for WP, specifically, the extension method GetResponseAsync() & friends for WebRequest).

Are you interested in merging this pull request? If not, please let me know why, and I'll try and improve the pull request with your comments in mind.

Thanks for your time,
David Hartveld

@dlhartveld

This comment has been minimized.

Show comment
Hide comment
@dlhartveld

dlhartveld Aug 9, 2013

Contributor

PS. In your other starter kit repository, there are also two instances; I can prepare a patch for those cases as well, if you are interested.

Contributor

dlhartveld commented Aug 9, 2013

PS. In your other starter kit repository, there are also two instances; I can prepare a patch for those cases as well, if you are interested.

ChrisKoenig added a commit that referenced this pull request Aug 22, 2013

Merge pull request #4 from dlhartveld/refactor-apm-to-aa
Replacement of APM usage (Begin/EndGetResponse, etc.) with TAP (GetResponceAsync) and async/await

@ChrisKoenig ChrisKoenig merged commit 64d0f03 into ChrisKoenig:master Aug 22, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment