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

Merged
merged 3 commits into from Aug 9, 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 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

@vortexwolf

This comment has been minimized.

Show comment
Hide comment
@vortexwolf

vortexwolf Aug 9, 2013

Owner

I'm familiar with new async/await keywords, though I never used them.
Ok, I will merge you changes and test how they work.

Owner

vortexwolf commented Aug 9, 2013

I'm familiar with new async/await keywords, though I never used them.
Ok, I will merge you changes and test how they work.

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