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

Removed unnecessary async/await keywords #4

Merged
merged 1 commit into from Sep 10, 2013

Conversation

Projects
None yet
2 participants
@semihokur
Contributor

semihokur commented Sep 8, 2013

I am a PhD student in the CS department at the University of Illinois. I'm currently doing research on asynchronous programming in phone applications. I developed a tool that automatically improves async/await usages by doing transformations.

The tool found some opportunities in your application. There was no need to use async/await for 3 methods. It will decrease the overhead that is caused by async/await state machine and it will simplify the code. Removing async/await does not change the behavior at all.

Are you interested in merging this pull request? If not, please let me know why.

Thanks for your time,

Bunk added a commit that referenced this pull request Sep 10, 2013

Merge pull request #4 from semihokur/master
Removed unnecessary async/await keywords

@Bunk Bunk merged commit 8e35018 into Bunk:master Sep 10, 2013

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