Skip to content

Rollup of 8 pull requests - #69804

Merged
bors merged 20 commits into
rust-lang:masterfrom
Centril:rollup-u86dc1g
Mar 8, 2020
Merged

Rollup of 8 pull requests#69804
bors merged 20 commits into
rust-lang:masterfrom
Centril:rollup-u86dc1g

Conversation

@Centril

Copy link
Copy Markdown
Contributor

Successful merges:

Failed merges:

r? @ghost

JohnTitorand others added 20 commits March 3, 2020 18:22
…estebank
resolve, inconsistent binding mode: tweak wording
Now that we can have e.g. `let Ok(x) | Err(x) = res;`, it's no longer appropriate to refer to "the same *match arm*", so let's tweak the wording.
r? @estebank
On mismatched delimiters, only point at empty blocks that are in the same line
We point at empty blocks when we have mismatched braces to detect cases where editors auto insert `}` after writing `{`. Gate this to only the case where the entire span is in the same line so we never point at explicitly empty blocks.
reduce test size for Miri
The larger sizes take quite a while, and there is probably little point in repeating this quite so often.
…opy, r=oli-obk
mir::Local is Copy we can pass it by value in these cases
r? @oli-obk
Add `Layout::dangling()` to return a well-aligned `NonNull<u8>`
Adds a convenient function to `Layout` to create a `NonNull<u8>` out of a layout to be returned on ZST allocations.
This is the first item on the roadmap to support ZSTs in `AllocRef`: rust-lang/wg-allocators#38 (comment)
r? @Amanieu
…mulacrum
Correct version that relaxed orphan rules
Docs say
> Prior to Rust 1.40, if the destination type was not part of the current crate then you couldn't implement From directly
Version 1.41 of Rust introduced the change to the orphan rules - https://blog.rust-lang.org/2020/01/30/Rust-1.41.0.html#relaxed-restrictions-when-implementing-traits - so this should say
> Prior to Rust 1.41...
@CentrilCentril added the rollup A PR which is a rollup label Mar 7, 2020
@Centril

Copy link
Copy Markdown
ContributorAuthor

@bors r+ p=1001 rollup=never

@bors

bors commented Mar 7, 2020

Copy link
Copy Markdown
Collaborator

📌 Commit 8ee7278 has been approved by Centril

@bors

bors commented Mar 7, 2020

Copy link
Copy Markdown
Collaborator

🌲 The tree is currently closed for pull requests below priority 1000, this pull request will be tested once the tree is reopened

@borsbors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Mar 7, 2020
@bors

bors commented Mar 7, 2020

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 8ee7278 with merge 703fc64f9fa09c3ea37080aa653ee359a9ba96ad...

@rust-highfive

Copy link
Copy Markdown
Contributor

Your PR failed (pretty log, raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @rust-lang/infra. (Feature Requests)

@bors

bors commented Mar 7, 2020

Copy link
Copy Markdown
Collaborator

💔 Test failed - checks-azure

@borsbors removed the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Mar 7, 2020
@rust-highfive

Copy link
Copy Markdown
Contributor

Your PR failed (pretty log, raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @rust-lang/infra. (Feature Requests)

@bors

bors commented Mar 7, 2020

Copy link
Copy Markdown
Collaborator

💔 Test failed - checks-azure

@borsbors 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 Mar 7, 2020
@Dylan-DPC-zz

Copy link
Copy Markdown

@bors treeclosed-

@Dylan-DPC-zz

Copy link
Copy Markdown

@bors retry

@borsbors 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 Mar 8, 2020
@bors

bors commented Mar 8, 2020

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 8ee7278 with merge 52f039cce3f6d141072be54c8146a33934567183...

@Dylan-DPC-zz

Copy link
Copy Markdown

@bors treeclosed=1000 retry

@bors

bors commented Mar 8, 2020

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 8ee7278 with merge da3b2b3305c35ce245cb80e4c326c8a038215178...

@rust-highfive

Copy link
Copy Markdown
Contributor

Your PR failed (pretty log, raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @rust-lang/infra. (Feature Requests)

@bors

bors commented Mar 8, 2020

Copy link
Copy Markdown
Collaborator

💔 Test failed - checks-azure

@borsbors 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 Mar 8, 2020
@Centril

Copy link
Copy Markdown
ContributorAuthor

@bors retry

@borsbors 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 Mar 8, 2020
@bors

bors commented Mar 8, 2020

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 8ee7278 with merge f943349...

@bors

bors commented Mar 8, 2020

Copy link
Copy Markdown
Collaborator

☀️ Test successful - checks-azure
Approved by: Centril
Pushing f943349 to master...

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-by-borsThis PR was explicitly merged by bors.rollupA PR which is a rollupS-waiting-on-borsStatus: 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.

11 participants

@Centril@bors@rust-highfive@Dylan-DPC-zz@JohnTitor@estebank@RalfJung@matthiaskrgr@spastorino@TimDiekmann@jongiddy