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

Minor documentation fixes. #24113

Merged
merged 2 commits into from
Apr 7, 2015
Merged

Minor documentation fixes. #24113

merged 2 commits into from
Apr 7, 2015

Conversation

mbrubeck
Copy link
Contributor

@mbrubeck mbrubeck commented Apr 6, 2015

  • Fix broken "module-level documentation" link on the trait Any docs and related broken markup on the std::any docs.
  • Remove an outdated or incorrect notice in the BufRead::lines docs. There is no such read_string function, and lines never returns an error.

r? @steveklabnik

There is no `read_string` function, and `lines` never returns an error.
///
/// [mod]: ../index.html
/// [mod]: index.html
Copy link
Member

Choose a reason for hiding this comment

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

Doesn't this break the link?

Copy link
Member

Choose a reason for hiding this comment

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

oh i see, nevermind

@steveklabnik
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Apr 6, 2015

📌 Commit 6e86c63 has been approved by steveklabnik

Manishearth added a commit to Manishearth/rust that referenced this pull request Apr 7, 2015
 * Fix broken \"module-level documentation\" link on the [`trait Any` docs](http://doc.rust-lang.org/std/any/trait.Any.html) and related broken markup on the [`std::any` docs](http://doc.rust-lang.org/std/any/index.html).

* Remove an outdated or incorrect notice in the `BufRead::lines` docs. There is no such `read_string` function, and `lines` never returns an error.

r? @steveklabnik
@bors bors merged commit 6e86c63 into rust-lang:master Apr 7, 2015
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