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

Rollup of 7 pull requests #46745

Merged
merged 14 commits into from
Dec 15, 2017
Merged

Rollup of 7 pull requests #46745

merged 14 commits into from
Dec 15, 2017

Conversation

matthewjasper and others added 14 commits December 9, 2017 13:50
The implementation was introduced when changing hash storage from
Unique to *mut, but it was changed back to Unique.
An invalid average now simply prints “N/A”. Fixes rust-lang#46725.
…=steveklabnik

Use a better link for method resolution in Deref docs

rust-lang/reference#149 breaks these links, so make them point to somewhere which won't break and provides a more deatailed description of method resolution.

cc @Havvy
r? @steveklabnik
Expose the line and column fields from the proc_macro::LineColumn struct

Right now the `LineColumn` struct is pretty useless because the fields are private.

This patch just marks the fields as public, which seems like the easiest solution.
Help Cargo tolerate RUSTFLAGS="--print=native-static-libs"

Alternative to rust-lang/cargo#4807

Having this "error" message was a mistake, as it's firing at exactly wrong time when Cargo is trying to read the output of other print commands.
Remove Sync and Send implementation for RawTable

The implementation was introduced when changing hash storage from Unique to *mut, but it was changed back to Unique.
Fix division-by-zero ICE in -Z perf-stats

An invalid average now simply prints “N/A”. Fixes rust-lang#46725.
@rust-highfive
Copy link
Collaborator

r? @KodrAus

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

@steveklabnik
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Contributor

bors commented Dec 15, 2017

📌 Commit b98de5d has been approved by steveklabnik

@bors
Copy link
Contributor

bors commented Dec 15, 2017

⌛ Testing commit b98de5d with merge 50f6c3e...

bors added a commit that referenced this pull request Dec 15, 2017
Rollup of 7 pull requests

- Successful merges: #46601, #46652, #46690, #46705, #46710, #46728, #46737
- Failed merges:
@bors
Copy link
Contributor

bors commented Dec 15, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: steveklabnik
Pushing 50f6c3e to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.