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 23 pull requests #35592

Merged
merged 56 commits into from
Aug 12, 2016
Merged

Rollup of 23 pull requests #35592

merged 56 commits into from
Aug 12, 2016

Conversation

Cengiz Can and others added 30 commits August 4, 2016 03:11
- Fix note message, fix tests.
Jonathan Turner added 7 commits August 11, 2016 06:34
Add tracking issue for `String::insert_str`
Remove redundant `&mut ref mut` in doc for Result::as_mut()

While a good example of how `&mut ref mut` is a no-op, I don't think we should show that here :)
See https://users.rust-lang.org/t/mnemonic-for-reading-qualifiers/6853

r? @steveklabnik
Fix docs typo in std::os::unix::net::SocketAddr::is_unnamed
…t, r=jonathandturner

E0072 update error format

Part of  rust-lang#35233

Fixes rust-lang#35506

r? @jonathandturner

The bonus for this issue currently seems to be impossible to do reliably, as the compiler seems to lack span information for item names alone, like `Foo` in `struct Foo { ... }`. It would be possible to hack something together by computing span offsets, but that seems like a solution that would be begging for trouble.

A proper solution to this would, of course, be to add span information to the right place (seems to be `rustc::hir::Item::name` but I may be wrong).
@sophiajt
Copy link
Contributor Author

@bors r+ p=1

@bors
Copy link
Contributor

bors commented Aug 11, 2016

📌 Commit cdedad5 has been approved by jonathandturner

@rust-highfive
Copy link
Collaborator

r? @brson

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

@bors
Copy link
Contributor

bors commented Aug 11, 2016

⌛ Testing commit cdedad5 with merge 0bf2360...

@bors
Copy link
Contributor

bors commented Aug 11, 2016

💔 Test failed - auto-win-msvc-64-opt-rustbuild

@sophiajt
Copy link
Contributor Author

@bors r+ p=1

@bors
Copy link
Contributor

bors commented Aug 11, 2016

📌 Commit d3af9a3 has been approved by jonathandturner

@bors
Copy link
Contributor

bors commented Aug 11, 2016

⌛ Testing commit d3af9a3 with merge 8787a12...

bors added a commit that referenced this pull request Aug 11, 2016
@bors
Copy link
Contributor

bors commented Aug 11, 2016

💔 Test failed - auto-linux-64-debug-opt

@sophiajt
Copy link
Contributor Author

Timed out...

@bors retry

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.