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

Doc backport #24814

Merged
merged 19 commits into from
Apr 25, 2015
Merged

Doc backport #24814

merged 19 commits into from
Apr 25, 2015

Conversation

steveklabnik and others added 19 commits April 25, 2015 12:06
Only backporting the style portion, as changing the test suite is needless churn, and the book/reference are getting their own imports at the end.
In 8f5b5f9, `default::Default` was
added to the prelude, so these imports are no longer necessary.
Without the `box` keyword, one of these two reasons is not correct, so
let's just eliminate this section and elaborate on the reason for the
legit use case inline.

Fixes rust-lang#24511
As far as I can tell, this conversion to integer to floating point does not need to happen and is beside the point
- rust-lang#1237 is closed
- `libgcc` is now statically linked:
  rust-lang#17471
@steveklabnik
Copy link
Member Author

@bors: try

@bors
Copy link
Contributor

bors commented Apr 25, 2015

🔒 Merge conflict

@rust-highfive
Copy link
Collaborator

r? @gankro

(rust_highfive has picked a reviewer for you, use r? to override)

@steveklabnik steveklabnik assigned steveklabnik and unassigned Gankra Apr 25, 2015
@steveklabnik
Copy link
Member Author

Ahh i guess try only works to master.

@steveklabnik
Copy link
Member Author

I'm gonna run a test locally before merging.

@Gankra
Copy link
Contributor

Gankra commented Apr 25, 2015

Presumably I don't actually need to review anything for this kind of PR?

@steveklabnik
Copy link
Member Author

@gankro yeah, I already assigned it to myself. ❤️

@steveklabnik
Copy link
Member Author

Passed all tests locally. 🎊

steveklabnik added a commit that referenced this pull request Apr 25, 2015
@steveklabnik steveklabnik merged commit 7ab793e into rust-lang:beta Apr 25, 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.