Skip to content

Elaborate a bit in the Reader docs regarding stream position. #6621

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

Merged
merged 2 commits into from
May 20, 2013

Conversation

steveklabnik
Copy link
Member

Had a conversation with @cmr in IRC about some places that these
docs were confusing. The functions that advance the stream now say so.

In addition, I think that calling out the similarities to familliar C
functions should help people coming from other languages.

Had a conversation with @cmr in IRC about some places that these
docs were confusing. The functions that advance the stream now say so.

In addition, I think that calling out the similarities to familliar C
functions should help people coming from other languages.
@brson
Copy link
Contributor

brson commented May 19, 2013

r+

@steveklabnik
Copy link
Member Author

@brson I think your r+ didn't go through because it was on the PR, not on the commit?

This is now 100% consistent. Whoops!
bors added a commit that referenced this pull request May 20, 2013
Had a conversation with @cmr in IRC about some places that these
docs were confusing. The functions that advance the stream now say so.

In addition, I think that calling out the similarities to familliar C
functions should help people coming from other languages.
@bors bors closed this May 20, 2013
@bors bors merged commit dcc2879 into rust-lang:incoming May 20, 2013
@steveklabnik steveklabnik deleted the expand_reader_docs branch October 25, 2017 18:25
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jan 30, 2021
…n_deref, r=flip1995

Improve the example in `ref_in_deref`

Add a suggested code to the example in doc

changelog: none
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.

4 participants