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

Clean up std::raw docs #35281

Merged
merged 1 commit into from
Aug 6, 2016
Merged

Clean up std::raw docs #35281

merged 1 commit into from
Aug 6, 2016

Conversation

apasel422
Copy link
Contributor

There is no longer a Repr trait, so mentioning a missing impl of it was potentially confusing.

r? @steveklabnik

@steveklabnik
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Aug 4, 2016

📌 Commit 33a4806 has been approved by steveklabnik

sophiajt pushed a commit to sophiajt/rust that referenced this pull request Aug 4, 2016
Clean up `std::raw` docs

There is no longer a `Repr` trait, so mentioning a missing impl of it was potentially confusing.

r? @steveklabnik
steveklabnik added a commit to steveklabnik/rust that referenced this pull request Aug 4, 2016
Clean up `std::raw` docs

There is no longer a `Repr` trait, so mentioning a missing impl of it was potentially confusing.

r? @steveklabnik
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Aug 5, 2016
Clean up `std::raw` docs

There is no longer a `Repr` trait, so mentioning a missing impl of it was potentially confusing.

r? @steveklabnik
bors added a commit that referenced this pull request Aug 5, 2016
@TimNN
Copy link
Contributor

TimNN commented Aug 5, 2016

A local build including this PR produced a linkchecker error: core/raw/struct.TraitObject.html:70: broken link - core/mem/fn.transmute.html

Which seems correct, since https://doc.rust-lang.org/nightly/core/mem/fn.transmute.html is a 404 as well.

@GuillaumeGomez
Copy link
Member

@bors: rollup-

@apasel422
Copy link
Contributor Author

Updated the link to point to the intrinsic instead.

bors added a commit that referenced this pull request Aug 5, 2016
@GuillaumeGomez
Copy link
Member

Please squash your commits.

There is no longer a `Repr` trait, so mentioning a missing impl of it
was potentially confusing.
@apasel422
Copy link
Contributor Author

Squashed.

@GuillaumeGomez
Copy link
Member

Thanks!

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Aug 6, 2016

📌 Commit da2b7a6 has been approved by GuillaumeGomez

eddyb added a commit to eddyb/rust that referenced this pull request Aug 6, 2016
Clean up `std::raw` docs

There is no longer a `Repr` trait, so mentioning a missing impl of it was potentially confusing.

r? @steveklabnik
@bors bors merged commit da2b7a6 into rust-lang:master Aug 6, 2016
@apasel422 apasel422 deleted the repr branch August 6, 2016 16:58
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.

5 participants