Change API for `spawn`/`start` to be just like `std::thread`

dpc / mioco.pre-0.9

Scalable, coroutine-based, asynchronous IO handling library for Rust programming language. (aka MIO COroutines).

Mozilla Public License 2.0

457 stars 30 forks source link

Closed dpc closed 8 years ago

dpc commented 8 years ago

closures should be of type FnOnce() -> T, the Result<T> return type is unnecessary
every closure should be able to return different T
mioco::spawn should return a JoinHandle
mioco::start should return just Result<T> of the starting function, just like JoinHandle::join().

dpc commented 8 years ago

@troplin @Drakulix

troplin commented 8 years ago

mioco::start should return a io::Result<T> IMO, as does JoinHandle::join() Otherwise, what would you return on panic or shutdown?

dpc commented 8 years ago

Right @troplin . Updated the description.

Drakulix commented 8 years ago

Seems reasonable. Although I don't mind the current design.

dpc commented 8 years ago

Wasn't that hard.