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

Improve documentation for std::io::LineWriter #27166

Merged
merged 1 commit into from
Jul 22, 2015

Conversation

steveklabnik
Copy link
Member

Beef up the struct docs, add examples for the methods.

r? @alexcrichton

/// (`0x0a`, `'\n'`) is detected.
///
/// The buffer will be written out when the writer is dropped.
/// The [`BufWriter`][bufwriter] struct wraps a writer and buffers its output.
/// But it only does this batched write when it goes out of scope. Sometimes,
Copy link
Member

Choose a reason for hiding this comment

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

This clause isn't quite true as it will write the internal buffer whenever it goes out of scope or it fills up the internal buffer (e.g. it's not infinitely buffered)

@steveklabnik steveklabnik force-pushed the doc_std_io_linewriter branch from 5e9129a to 94b8f28 Compare July 21, 2015 17:39
@steveklabnik
Copy link
Member Author

updated!

@alexcrichton
Copy link
Member

@bors: r+ 94b8f28

@alexcrichton
Copy link
Member

@bors: rollup

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Jul 21, 2015
… r=alexcrichton

Beef up the struct docs, add examples for the methods.

r? @alexcrichton
steveklabnik added a commit to steveklabnik/rust that referenced this pull request Jul 21, 2015
… r=alexcrichton

Beef up the struct docs, add examples for the methods.

r? @alexcrichton
steveklabnik added a commit to steveklabnik/rust that referenced this pull request Jul 22, 2015
… r=alexcrichton

Beef up the struct docs, add examples for the methods.

r? @alexcrichton
steveklabnik added a commit to steveklabnik/rust that referenced this pull request Jul 22, 2015
… r=alexcrichton

Beef up the struct docs, add examples for the methods.

r? @alexcrichton
@steveklabnik steveklabnik force-pushed the doc_std_io_linewriter branch from 94b8f28 to e31ad95 Compare July 22, 2015 14:19
Beef up the struct docs, add examples for the methods.
@steveklabnik
Copy link
Member Author

@bors: r=alexcrichton rollup

@bors
Copy link
Contributor

bors commented Jul 22, 2015

📌 Commit e31ad95 has been approved by alexcrichton

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Jul 22, 2015
… r=alexcrichton

Beef up the struct docs, add examples for the methods.

r? @alexcrichton
steveklabnik added a commit to steveklabnik/rust that referenced this pull request Jul 22, 2015
… r=alexcrichton

Beef up the struct docs, add examples for the methods.

r? @alexcrichton
@bors bors merged commit e31ad95 into rust-lang:master Jul 22, 2015
@steveklabnik steveklabnik deleted the doc_std_io_linewriter branch June 19, 2016 20:33
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.

3 participants