Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upDefine Mio's behavior #307
Comments
This comment has been minimized.
This comment has been minimized.
|
Other things:
EDIT: Now included in the top post. |
carllerche
added
the
api
label
Dec 1, 2015
carllerche
added this to the v1.0 milestone
Dec 1, 2015
This comment has been minimized.
This comment has been minimized.
|
Also, given the recent changes for tracking the event loop a socket is registered on, do we care if mio's io objects are |
This comment has been minimized.
This comment has been minimized.
|
We probably do want them to be |
This comment has been minimized.
This comment has been minimized.
|
Idea: type encode socket registration state: https://www.reddit.com/r/rust/comments/3vdz26/mio_v05_released_now_with_windows_support/cxn8p8t |
This comment has been minimized.
This comment has been minimized.
|
Do we care about defining the behavior of mio under |
This comment has been minimized.
This comment has been minimized.
danburkert
commented
Dec 14, 2015
|
Another thing:
At least for writing, it works fine on Linux. On OS X you are liable to get an |
This comment has been minimized.
This comment has been minimized.
Yeah.... not for 1.0 :) especially since that is a *nix specific use case. Do you happen to know what happens now? |
carllerche commentedNov 23, 2015
•
edited
Exact mio behavior should be documented and consistent across platforms.
A few cases to document:
TcpStreamis registered with anEventLoop, theEventLoopis closed, a new one is created and the socket is registered with the new one.TcpStreamis registered, then deregistered, thenregisteredwith a differentEventLoopEventLoopvalues.EventSet::none()EventSetTryRead&TryWritetake&mut selfor&self?Pollbe public and what should the API be. CanIoEventbe renamed toEvent.kqueueimplementationEventLoopthat does not own it.Durationfor timeouts?SyncandSendAtomicUsizeon Unix with compare and swap + relaxed