Skip to content
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

Replace ConfigureAwait(false) with ConfigureAwait(ben) #83

Open
danielmarbach opened this issue Mar 24, 2017 · 5 comments
Open

Replace ConfigureAwait(false) with ConfigureAwait(ben) #83

danielmarbach opened this issue Mar 24, 2017 · 5 comments
Assignees
Projects
Milestone

Comments

@danielmarbach
Copy link

I was exploring the code and came to the conclusion that YOU are doing it wrong.

With the introduction of #66 the code will be sprinkled with ConfigureAwait(false) but that is totally wrong! We should stop capturing Ben instead of the context! So I suggest we introduce ConfigureAwait(ben). Thoughts?

@Scooletz
Copy link

I used it once. The continuation was dispatched on the IOBP (IOBenPort) thread. Are you sure you want to do it @danielmarbach ?

@danielmarbach
Copy link
Author

Of course! Here is my rough draft

public async Ben Awesome(string itis) {
   await Food().ConfigureAwait(ben);
}

It will be using the Ben Continuation model. It will also be possible to write arbitrary ben-like types.

Everything will be dispatched with the BenScheduler. Paging in @stephentoub and @ljw1004 to get this going!

@danielmarbach
Copy link
Author

I just had an amazing revelation. With arbitrary ben-like types, I can write System.Daniel, write an arbitrary ben-like type and then

I CAN BECOME BEN!

MUAHAHAHAHA

@benaadams benaadams added this to the 1.1.0 milestone Mar 24, 2017
@benaadams benaadams added this to Ready in 1.1.0 Mar 24, 2017
@karelz
Copy link

karelz commented Mar 25, 2017

I think we need anonymous Ben types (we can start with his alter ego ;-))

@benaadams
Copy link
Owner

we can start with his alter ego

Mild mannered reporter ;-)

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

No branches or pull requests

4 participants