Skip to content

Rollup of 7 pull requests#45981

Merged
bors merged 17 commits intorust-lang:masterfrom
GuillaumeGomez:rollup
Nov 14, 2017
Merged

Rollup of 7 pull requests#45981
bors merged 17 commits intorust-lang:masterfrom
GuillaumeGomez:rollup

Conversation

@GuillaumeGomez
Copy link
Copy Markdown
Member Author

@bors: r+ p=1

@bors
Copy link
Copy Markdown
Collaborator

bors commented Nov 14, 2017

📌 Commit 0e19c4f has been approved by GuillaumeGomez

@bors
Copy link
Copy Markdown
Collaborator

bors commented Nov 14, 2017

⌛ Testing commit 0e19c4f with merge 2711f21...

bors added a commit that referenced this pull request Nov 14, 2017
Rollup of 7 pull requests

- Successful merges: #45815, #45941, #45950, #45951, #45961, #45967, #45970
- Failed merges:
@bors
Copy link
Copy Markdown
Collaborator

bors commented Nov 14, 2017

💔 Test failed - status-travis

@GuillaumeGomez
Copy link
Copy Markdown
Member Author

HTTP error it seems?

@bors: retry

@GuillaumeGomez
Copy link
Copy Markdown
Member Author

Woups hold on.

@bors: r-

…s, r=GuillaumeGomez

rustdoc: tweak notes on ignore/compile_fail examples

Part of rust-lang#44927

This is a softening of these notices to mention *why* a given example has a given callout, rather then telling viewers to be careful with an example. It also changes the character used for these samples from a warning logo to a circle-i/information logo.

![image](https://user-images.githubusercontent.com/5217170/32464361-5fbb5d9e-c305-11e7-8482-ce71b97a54df.png)
update match-expressions.rs with DepNode labels

As a part of rust-lang#44924, I have updated the match-expressions.rs. The PR has tests verified for the following dependency nodes for let-expressions

- MirValidated
- MirOptimized
- TypeCheckTables
- TypeOfItem
- GenericsOfItem
- PredicatesOfItem
- FnSignature

cc @michaelwoerister
r? @nikomatsakis
…rs-test-to-use-incr-except, r=michaelwoerister

incr: Make `unary_and_binary_exprs.rs` use `except`-style incremental checking

Part of rust-lang#44924

r? @michaelwoerister
…rk-Simulacrum

Use #!/usr/bin/env as shebang for Bash scripts

On some systems, the bash command could be available in another
directory than /bin. As such, to offer an env shebang is more
convenient.

This make sense even for docker scripts, as you can use Docker
on FreeBSD or SmartOS for example.
…rielb1

MIR-borrowck: don't ICE for cannot move from array error

Closes rust-lang#45694
compile-fail test E0508 now gives
```text
error[E0508]: cannot move out of type `[NonCopy; 1]`, a non-copy array (Ast)
  --> .\src\test\compile-fail\E0508.rs:18:18
   |
18 |     let _value = array[0];  //[ast]~ ERROR E0508
   |                  ^^^^^^^^
   |                  |
   |                  cannot move out of here
   |                  help: consider using a reference instead: `&array[0]`

error[E0508]: cannot move out of type `[NonCopy; 1]`, a non-copy array (Mir)
  --> .\src\test\compile-fail\E0508.rs:18:18
   |
18 |     let _value = array[0];  //[ast]~ ERROR E0508
   |                  ^^^^^^^^ cannot move out of here

error: aborting due to 2 previous errors
```
…eveklabnik

Fixed several pulldown warnings when documenting libstd.
@GuillaumeGomez
Copy link
Copy Markdown
Member Author

@bors: r+ p=1

@bors
Copy link
Copy Markdown
Collaborator

bors commented Nov 14, 2017

📌 Commit 66d0537 has been approved by GuillaumeGomez

@bors
Copy link
Copy Markdown
Collaborator

bors commented Nov 14, 2017

⌛ Testing commit 66d0537 with merge dbeb5bf...

bors added a commit that referenced this pull request Nov 14, 2017
Rollup of 7 pull requests

- Successful merges: #45815, #45941, #45950, #45961, #45967, #45970, #45977
- Failed merges:
@bors
Copy link
Copy Markdown
Collaborator

bors commented Nov 14, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: GuillaumeGomez
Pushing dbeb5bf to master...

@bors bors merged commit 66d0537 into rust-lang:master Nov 14, 2017
@GuillaumeGomez GuillaumeGomez deleted the rollup branch November 14, 2017 21:31
@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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants