Stepan Koltsov

> How come the event loop is dropping early for you? I can think of two scenarios: - Simply panic or logic error or whatever. Program should properly terminate even...

> The Loop::new() function is quite expensive in terms of resource allocation, so it's not intended to be called once per client. Suppose, your client is a command line database...

> If that's destroyed, the the channel has no ability to actually block. Channel should not block, it should immediately return an error. > What you'd want to use in...

Public `into_vec` is probably not needed when `From` is implemented for `Vec`.

I think * `into_vec` can be added any time later, but it can't be removed because of backward compatibility * and `into_vec` may give users wrong idea that operation is...

> In general though you don't know whether the async libraries you're calling are going to be spawning tasks For me that's the reason for this patch, not against. When...

> If we added a way to explicitly spawn a "background task" then we would be able to land this. Doesn't `Handle::spawn` spawn a background task?

What's the difference?

What should be a result of `drain`?

`Vec::drain` returns an iterator. Do you think `Bytes::drain` should return an iterator too?