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.

Sign up for GitHub

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

Jump to bottom

io::Seek should implement .tell() #26650

Closed
whitequark opened this issue Jun 29, 2015 · 3 comments
Closed

io::Seek should implement .tell() #26650

whitequark opened this issue Jun 29, 2015 · 3 comments
Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one.

Comments

@whitequark
Copy link
Member

A file.tell() is much nicer than file.seek(std::io::SeekFrom::Current(0)). Additionally, I think .tell() doesn't need the file to be mutably borrowed (but I'm not sure).
@steveklabnik steveklabnik added A-libs C-enhancement Category: An issue proposing an enhancement or a PR with one. labels Jun 29, 2015
@alexcrichton
Copy link
Member

Thanks for the report! Additions to the standard I/O traits, however, should go through the RFC process, so this issue should go into rust-lang/rust instead of this repo.

@alexcrichton
Copy link
Member

Er, rust-lang/rfcs*

@whitequark
Copy link
Member Author

And... you're just closing? That's not a very nice way to handle this...

Anyway, rust-lang/rfcs#1181.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@steveklabnik @whitequark @alexcrichton