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

Tighten span for E0063 #33517

Merged
merged 1 commit into from
May 15, 2016
Merged

Tighten span for E0063 #33517

merged 1 commit into from
May 15, 2016

Conversation

sanxiyn
Copy link
Member

@sanxiyn sanxiyn commented May 9, 2016

No description provided.

@rust-highfive
Copy link
Collaborator

r? @jroesch

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

@sanxiyn
Copy link
Member Author

sanxiyn commented May 9, 2016

struct Point {
    x: i32,
    y: i32,
}

fn main() {
    Point {
        x: 1,
    };
}

Before:

error: missing field `y` in initializer of `Point` [--explain E0063]
 --> test.rs:7:5
7 |>     Point {
  |>     ^

After:

error: missing field `y` in initializer of `Point` [--explain E0063]
 --> test.rs:7:5
7 |>     Point {
  |>     ^^^^^

@nagisa
Copy link
Member

nagisa commented May 10, 2016

r? @nikomatsakis or @jonathandturner

@nagisa
Copy link
Member

nagisa commented May 10, 2016

Actually, lemme do this:

@bors r+.

@nikomatsakis
Copy link
Contributor

lgtm!

@nagisa
Copy link
Member

nagisa commented May 10, 2016

@bors r+

@bors
Copy link
Contributor

bors commented May 10, 2016

📌 Commit f6392a2 has been approved by nagisa

@eddyb
Copy link
Member

eddyb commented May 11, 2016

@bors rollup

@bors
Copy link
Contributor

bors commented May 11, 2016

☔ The latest upstream changes (presumably #33425) made this pull request unmergeable. Please resolve the merge conflicts.

bors added a commit that referenced this pull request May 11, 2016
Rollup of 9 pull requests

- Successful merges: #33129, #33260, #33345, #33386, #33522, #33524, #33528, #33539, #33542
- Failed merges: #33342, #33475, #33517
bors added a commit that referenced this pull request May 11, 2016
Rollup of 3 pull requests

- Successful merges: #33401, #33489, #33558
- Failed merges: #33342, #33475, #33517
@sanxiyn
Copy link
Member Author

sanxiyn commented May 12, 2016

@bors r=nagisa rollup

@bors
Copy link
Contributor

bors commented May 12, 2016

📌 Commit a7902b1 has been approved by nagisa

eddyb added a commit to eddyb/rust that referenced this pull request May 12, 2016
bors added a commit that referenced this pull request May 12, 2016
eddyb added a commit to eddyb/rust that referenced this pull request May 13, 2016
sanxiyn added a commit to sanxiyn/rust that referenced this pull request May 14, 2016
bors added a commit that referenced this pull request May 14, 2016
Rollup of 15 pull requests

- Successful merges: #33342, #33393, #33415, #33475, #33517, #33533, #33534, #33565, #33580, #33590, #33591, #33598, #33603, #33604, #33605
- Failed merges: #33578
Manishearth added a commit to Manishearth/rust that referenced this pull request May 15, 2016
bors added a commit that referenced this pull request May 15, 2016
@bors bors merged commit a7902b1 into rust-lang:master May 15, 2016
bors added a commit that referenced this pull request May 16, 2016
@sanxiyn sanxiyn deleted the tight-span branch May 16, 2016 03:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants