This repository has been archived by the owner. It is now read-only.

Removed unnecessary async/await keywords #21

Merged
merged 1 commit into from Aug 25, 2013

Conversation

Projects
None yet
2 participants
@semihokur
Contributor

semihokur commented Aug 25, 2013

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

@ghost ghost assigned PedroLamas Aug 25, 2013

PedroLamas added a commit that referenced this pull request Aug 25, 2013

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

@PedroLamas PedroLamas merged commit 5641a5c into Cimbalino:master Aug 25, 2013

@PedroLamas

This comment has been minimized.

Show comment
Hide comment
@PedroLamas

PedroLamas Aug 25, 2013

Member

Totally agree, I normally try to take the same minimizing approach, though it seems I missed these (and probably more!). Thanks!

Member

PedroLamas commented Aug 25, 2013

Totally agree, I normally try to take the same minimizing approach, though it seems I missed these (and probably more!). Thanks!

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Sep 29, 2013

@semihokur Any news about your app?

ghost commented Sep 29, 2013

@semihokur Any news about your app?

@semihokur

This comment has been minimized.

Show comment
Hide comment
@semihokur

semihokur Oct 2, 2013

Contributor

Hi, please visit our webpage and subscribe our list to get informed when we release the toolkit.

http://LearnAsync.NET

Contributor

semihokur commented Oct 2, 2013

Hi, please visit our webpage and subscribe our list to get informed when we release the toolkit.

http://LearnAsync.NET

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