Improved async/await usage #1

Merged
merged 2 commits into from Sep 10, 2013

Conversation

Projects
None yet
2 participants
@semihokur
Contributor

semihokur commented Sep 9, 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 2 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.

Second, it added corresponding async calls of methods in JsonConvert class under "async" declared methods. For instance, it replaces "JsonConvert.DeserializeObject" with "JsonConvert.DeserializeObjectAsync".

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

Thanks for your time,

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

Merge pull request #1 from semihokur/master
Improved async/await usage

@jarvisji jarvisji merged commit 21e3775 into jarvisji:master Sep 10, 2013

@jarvisji

This comment has been minimized.

Show comment
Hide comment
@jarvisji

jarvisji Sep 10, 2013

Owner

Thanks for your advice!

Owner

jarvisji commented Sep 10, 2013

Thanks for your advice!

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