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

Add a io::Writer wrapper that supports pushback #4031

Closed
erickt opened this issue Nov 24, 2012 · 7 comments
Closed

Add a io::Writer wrapper that supports pushback #4031

erickt opened this issue Nov 24, 2012 · 7 comments
Labels
P-low Low priority

Comments

@erickt
Copy link
Contributor

erickt commented Nov 24, 2012

C's stdio has a function ungetc that lets you push back a single char on a stream. While we could expose support for the c functions, this wouldn't help with our str/bytes writers. Instead, how about we make a trait/impl io::Writer wrapper that supports lookahead and pushback for any io::Writers?
@erickt erickt mentioned this issue Nov 24, 2012
Closed
catamorphism added a commit that referenced this issue Nov 24, 2012
@catamorphism
[libs] Remove unread_byte method from core::io::Reader
455d73c 
Method isn't used. See discussion on #2738 and #4031

Closes #2738
@bstrie
Copy link
Contributor

bstrie commented May 28, 2013

Nominating for Maturity 5, Production Ready

@graydon
Copy link
Contributor

graydon commented Jul 18, 2013

sub-bug of #4248, and a feature

@graydon
Copy link
Contributor

graydon commented Jul 18, 2013

accepted for feature-complete milestone

@pnkfelix
Copy link
Member

assigning P-low.

@sfackler
Copy link
Member

Shouldn't this be an io::Reader wrapper?

@pongad
Copy link
Contributor

pongad commented Mar 3, 2014

Is something like this what everyone has in mind? https://gist.github.com/pongad/9316641
I can open a PR if everyone likes it ok.

@steveklabnik
Copy link
Member

I'm pulling a massive triage effort to get us ready for 1.0. As part of this, I'm moving stuff that's wishlist-like to the RFCs repo, as that's where major new things should get discussed/prioritized.

This issue has been moved to the RFCs repo: rust-lang/rfcs#856

@steveklabnik steveklabnik mentioned this issue Feb 15, 2015
Open
RalfJung added a commit to RalfJung/rust that referenced this issue Dec 8, 2024
@RalfJung
Merge pull request rust-lang#4031 from devnexen/solarish_stat
987d642 
filesystem support for solarish: stat
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P-low Low priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
8 participants
@graydon @steveklabnik @erickt @pnkfelix @bstrie @sfackler @thestinger @pongad