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 13 pull requests #54711

Merged
merged 37 commits into from
Oct 1, 2018
Merged

Rollup of 13 pull requests #54711

merged 37 commits into from
Oct 1, 2018

Conversation

kennytm
Copy link
Member

@kennytm kennytm commented Oct 1, 2018

Successful merges:

tbu- and others added 30 commits September 4, 2018 11:35
Copy the documentation over to `slice::from_raw_parts_mut`.
Hitting this branch in Clippy and I think it makes sense to print
both values here in case other people hit this branch, too.
Currently we ship sanitizer libraries as they're built, but these names
unfortunately conflict with the names of the sanitizer libraries
installed on the system. If a crate, for example, links in C code that
wants to use the system sanitizer and the Rust code doesn't use
sanitizers at all, then using `cargo` may accidentally pull in the
Rust-installed sanitizer library due to a conflict in names.

This change is intended to be entirely transparent for Rust users of
sanitizers, it should only hopefully improve our story with other users!

Closes rust-lang#54134
…Jung

Document that slices cannot be larger than `isize::MAX` bytes

Fixes rust-lang#53676.
Better user experience when attempting to call associated functions with dot notation

Closes rust-lang#22692
…ed_crate, r=estebank

in which we include attributes in unused `extern crate` suggestion spans

![unused_extern](https://user-images.githubusercontent.com/1076988/45921698-50243e80-be6f-11e8-930a-7b2a33b4935c.png)

Resolves rust-lang#54400.

r? @estebank
…eGomez

Indicate how to move value out of Box in docs.

Fixes rust-lang#53634.
…, r=estebank

Added help message for `impl_trait_in_bindings` feature gate

r? @estebank

CC @Centril
A few cleanups and minor improvements to rustc/infer

- use unwrap_or(_else) where applicable
- convert single-branch matches to if-let
- use to_owned instead of to_string with string literals
- improve vector allocations
- readability improvements
- miscellaneous minor code improvements
Correct doc for WorkQueue<T>::pop().

The old function doc looks like copy-pasta from WorkQueue::insert().

WorkQueue::pop() does not enqueue nor does it return a boolean false.  Doc corrected accordingly.
…hewjasper

Improve bug! message for impossible case in Relate

Hitting this branch [in Clippy][clippy_issue] and I think it makes sense to print
both values here in case other people hit this branch, too.

(still have to figure out why this branch is hit)

[clippy_issue]: rust-lang/rust-clippy#2831 (comment)
Rename sanitizer runtime libraries on OSX

Currently we ship sanitizer libraries as they're built, but these names
unfortunately conflict with the names of the sanitizer libraries
installed on the system. If a crate, for example, links in C code that
wants to use the system sanitizer and the Rust code doesn't use
sanitizers at all, then using `cargo` may accidentally pull in the
Rust-installed sanitizer library due to a conflict in names.

This change is intended to be entirely transparent for Rust users of
sanitizers, it should only hopefully improve our story with other users!

Closes rust-lang#54134
Make ./x.py help <cmd> invoke ./x.py <cmd> -h on its own

Fixes rust-lang#49475

r? @Mark-Simulacrum
@kennytm
Copy link
Member Author

kennytm commented Oct 1, 2018

@bors r+ p=10

@bors
Copy link
Contributor

bors commented Oct 1, 2018

📌 Commit 3401db0 has been approved by kennytm

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Oct 1, 2018
@bors
Copy link
Contributor

bors commented Oct 1, 2018

⌛ Testing commit 3401db0 with merge 1e2f78ac02a71a7edd1985c3daaa1fbb99000377...

@bors
Copy link
Contributor

bors commented Oct 1, 2018

💔 Test failed - status-travis

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Oct 1, 2018
@rust-highfive

This comment has been minimized.

@kennytm
Copy link
Member Author

kennytm commented Oct 1, 2018

@bors retry

An error occurred while generating the build script.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 1, 2018
…shearth

Add nightly check for tool_lints warning

cc rust-lang#54358

Let's add the nightly check, just in case something prevents a stabilization until 1.31.beta.
…abling-ast-check, r=alexcrichton

Remove `-Z disable_ast_check_for_mutation_in_guard`

One should use `#![feature(bind_by_move_pattern_guards)]` over `-Z disable_ast_check_for_mutation_in_guard`

cc rust-lang#15287
@kennytm kennytm changed the title Rollup of 10 pull requests Rollup of 13 pull requests Oct 1, 2018
@kennytm
Copy link
Member Author

kennytm commented Oct 1, 2018

@bors r+

Added #54674, #54713, #54676.

@bors
Copy link
Contributor

bors commented Oct 1, 2018

📌 Commit 5b08200 has been approved by kennytm

@bors
Copy link
Contributor

bors commented Oct 1, 2018

⌛ Testing commit 5b08200 with merge 6188c58...

bors added a commit that referenced this pull request Oct 1, 2018
Rollup of 13 pull requests

Successful merges:

 - #53784 (Document that slices cannot be larger than `isize::MAX` bytes)
 - #54308 (Better user experience when attempting to call associated functions with dot notation)
 - #54488 (in which we include attributes in unused `extern crate` suggestion spans)
 - #54544 (Indicate how to move value out of Box in docs.)
 - #54623 (Added help message for `impl_trait_in_bindings` feature gate)
 - #54641 (A few cleanups and minor improvements to rustc/infer)
 - #54656 (Correct doc for WorkQueue<T>::pop().)
 - #54674 (update miri)
 - #54676 (Remove `-Z disable_ast_check_for_mutation_in_guard`)
 - #54679 (Improve bug! message for impossible case in Relate)
 - #54681 (Rename sanitizer runtime libraries on OSX)
 - #54708 (Make ./x.py help <cmd> invoke ./x.py <cmd> -h on its own)
 - #54713 (Add nightly check for tool_lints warning)
@bors
Copy link
Contributor

bors commented Oct 1, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: kennytm
Pushing 6188c58 to master...

@bors bors merged commit 5b08200 into rust-lang:master Oct 1, 2018
@rust-highfive
Copy link
Collaborator

📣 Toolstate changed by #54711!

Tested on commit 6188c58.
Direct link to PR: #54711

🎉 miri on windows: build-fail → test-pass.
🎉 miri on linux: build-fail → test-pass.

rust-highfive added a commit to rust-lang-nursery/rust-toolstate that referenced this pull request Oct 1, 2018
Tested on commit rust-lang/rust@6188c58.
Direct link to PR: <rust-lang/rust#54711>

🎉 miri on windows: build-fail → test-pass.
🎉 miri on linux: build-fail → test-pass.
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
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 S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.