Unnecessary use of async/await in 2.0 sources

May 11, 2015 at 5:50 PM

Was looking into the 2.0 source code, and I noticed that most ResolveAsync/SendMessageAsync methods on DnsClient are awaiting instead of just returning tasks, even though they just await to return the value.

This adds some unnecessary overhead, as most methods are creating continuations just to hand the result to the caller. Most of the methods on DnsClient don't need to be async.

I'd create a PR if that was possible.