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

expose IoSlice, IoSliceMut #200

Merged
merged 1 commit into from
Sep 16, 2019
Merged

expose IoSlice, IoSliceMut #200

merged 1 commit into from
Sep 16, 2019

Conversation

yoshuawuyts
Copy link
Contributor

Exposes io::IoSlice and io::IoSliceMut. Given we're returning these from read_vectored and write_vectored it might make sense to just include them as part of our re-exports. Thanks!

Ref #131.

Signed-off-by: Yoshua Wuyts <yoshuawuyts@gmail.com>
@ghost
Copy link

ghost commented Sep 16, 2019

bors r+

bors bot added a commit that referenced this pull request Sep 16, 2019
190: Clean up the fs module and a few other places r=stjepang a=stjepang

Just a cleanup for various pieces of documentation, mainly around the `lib.rs` docs, the prelude, and the `fs` module. Some small bugs are also fixed along the way.

This PR is the first one in a series of review PRs that I will be submitting before the upcoming 1.0 release.

200: expose IoSlice, IoSliceMut r=stjepang a=yoshuawuyts

Exposes `io::IoSlice` and `io::IoSliceMut`. Given we're returning these from `read_vectored` and `write_vectored` it might make sense to just include them as part of our re-exports. Thanks!

Ref #131.

Co-authored-by: Stjepan Glavina <stjepang@gmail.com>
Co-authored-by: Yoshua Wuyts <yoshuawuyts@gmail.com>
@bors
Copy link
Contributor

bors bot commented Sep 16, 2019

Build succeeded

  • continuous-integration/travis-ci/push

@bors bors bot merged commit ab112e9 into master Sep 16, 2019
@yoshuawuyts yoshuawuyts deleted the expose-io-slice branch September 16, 2019 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant