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

fs: impl File::read_exact_at and File::write_all_at #44

Merged
merged 3 commits into from
Nov 5, 2022

Conversation

songzhi
Copy link
Contributor

@songzhi songzhi commented Aug 9, 2021

Current API and abstractions make it possible to write these helper methods.

Fixes #41

Copy link
Member

@carllerche carllerche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks. I had one thought inline.

@tsturzl
Copy link

tsturzl commented Sep 22, 2021

Just giving this a bump. I'm really interested in using the feature.

@songzhi songzhi force-pushed the songzhi/issue41 branch 2 times, most recently from d1997ae to ba77bb8 Compare April 13, 2022 06:02
@songzhi songzhi requested a review from carllerche April 13, 2022 06:04
@songzhi songzhi requested review from jiangliu and removed request for jiangliu August 11, 2022 02:39
@Noah-Kennedy
Copy link
Contributor

@songzhi can you resolve the conflicts? I'll merge once that's done.

Current API and abstractions make it possible to write these helper
methods.
Copy link
Collaborator

@FrankReh FrankReh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to say, let's try to get this committed. It's been out a long time.

@FrankReh
Copy link
Collaborator

@carllerche Waiting for your okay. You had requested changes so the page wants your blessing.

@FrankReh
Copy link
Collaborator

If @Noah-Kennedy and @carllerche say okay (actually I think Noah did already) we just need the branch rebased from master. Does the OP want to do that or would they like me to?

@Noah-Kennedy Noah-Kennedy merged commit a904aa9 into tokio-rs:master Nov 5, 2022
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.

Impl File::read_exact_at and File::write_all_at
8 participants