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

Replacement of APM usage with TAP and async/await #1

Merged
merged 3 commits into from Aug 27, 2013

Conversation

Projects
None yet
2 participants
@semihokur
Contributor

semihokur commented Aug 22, 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.

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

I also added a dependency that is needed: Microsoft.Bcl.Async by Microsoft. This package improves support for async/await-based programming for older frameworks, or frameworks that are missing functionality.

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,
Semih Okur

@semihokur

This comment has been minimized.

Show comment
Hide comment
@semihokur

semihokur Aug 27, 2013

Contributor

It should be fine now. Thanks!

Contributor

semihokur commented Aug 27, 2013

It should be fine now. Thanks!

PawelStroinski added a commit that referenced this pull request Aug 27, 2013

Merge pull request #1 from semihokur/master
Replacement of APM usage with TAP and async/await

@PawelStroinski PawelStroinski merged commit 2f5cfa3 into PawelStroinski:master Aug 27, 2013

@PawelStroinski

This comment has been minimized.

Show comment
Hide comment
@PawelStroinski

PawelStroinski Aug 27, 2013

Owner

Thank you Semih.

Owner

PawelStroinski commented Aug 27, 2013

Thank you Semih.

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