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

Merged
merged 3 commits into from Aug 22, 2013

Conversation

Projects
None yet
2 participants
@dlhartveld
Contributor

dlhartveld commented Aug 12, 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 three calls to APM methods.

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

These two commit contain the actual code-level changes:

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, TAP extension methods 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

wmeints pushed a commit that referenced this pull request Aug 22, 2013

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

@wmeints wmeints merged commit a4cbad7 into wmeints:master Aug 22, 2013

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