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 5 pull requests #64127

Merged
merged 13 commits into from
Sep 3, 2019
Merged

Rollup of 5 pull requests #64127

merged 13 commits into from
Sep 3, 2019

Conversation

Centril
Copy link
Contributor

@Centril Centril commented Sep 3, 2019

Successful merges:

Failed merges:

r? @ghost

estebank and others added 13 commits August 31, 2019 18:34
An initial refactoring before working on rust-lang#61809.

This moves the whole block into a method so that it can be reused in the
annotate-snippet output. It's already used in the new emitter, but
there's no UI tests with suggestions included in this PR.

A first look at some UI tests with suggestions showed that there's some
more work to do in [annotate-snippet-rs][annotate-snippet-rs] before the
new output is closer to the current one.
Emit a single error on if expr with expectation and no else clause

Fix rust-lang#60254.

r? @Centril
…ntril

Account for arbitrary self types in E0599

Fix rust-lang#62373
…bank

librustc_errors: Extract sugg/subst handling into method

An initial refactoring before working on rust-lang#61809.

This moves the whole block into a method so that it can be reused in the
annotate-snippet emitter. The method is already used in the new emitter, but
there's no UI tests with suggestions included in this PR.

A first look at some UI tests with suggestions showed that there's some
more work to do in [annotate-snippet-rs][annotate-snippet-rs] before the new output is closer to the
current one, so I opted to do that in a second step.

r? @estebank

[annotate-snippet-rs]: https://github.com/rust-lang/annotate-snippets-rs
use just one name when parameters and fields are the same
…e, r=estebank

Emit error on intrinsic to fn ptr casts

I'm not sure if a type error is the best way of doing this but it seemed like a relatively correct place to do it, and I expect this is a pretty rare case to hit anyway.

Fixes rust-lang#15694
@Centril
Copy link
Contributor Author

Centril commented Sep 3, 2019

@bors r+ p=5 rollup=never

@bors
Copy link
Contributor

bors commented Sep 3, 2019

📌 Commit 24e3b1d has been approved by Centril

@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 Sep 3, 2019
@bors
Copy link
Contributor

bors commented Sep 3, 2019

⌛ Testing commit 24e3b1d with merge b9de4ef...

bors added a commit that referenced this pull request Sep 3, 2019
Rollup of 5 pull requests

Successful merges:

 - #64049 (Emit a single error on if expr with expectation and no else clause)
 - #64056 (Account for arbitrary self types in E0599)
 - #64058 (librustc_errors: Extract sugg/subst handling into method)
 - #64071 (use just one name when parameters and fields are the same)
 - #64104 (Emit error on intrinsic to fn ptr casts)

Failed merges:

r? @ghost
@bors
Copy link
Contributor

bors commented Sep 3, 2019

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

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Sep 3, 2019
@bors bors merged commit 24e3b1d into rust-lang:master Sep 3, 2019
@Centril Centril deleted the rollup-dfgb9h8 branch September 3, 2019 23:11
@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
merged-by-bors This PR was explicitly merged by bors. 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.

5 participants