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

Add test for correct span for type #38606

Merged
merged 1 commit into from
Jan 10, 2017
Merged

Conversation

estebank
Copy link
Contributor

Adds test for and closes #27522.

@rust-highfive
Copy link
Collaborator

r? @brson

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

@petrochenkov
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Jan 1, 2017

📌 Commit 3caf478 has been approved by petrochenkov

@bors
Copy link
Contributor

bors commented Jan 1, 2017

⌛ Testing commit 3caf478 with merge 809de12...

@bors
Copy link
Contributor

bors commented Jan 1, 2017

💔 Test failed - status-travis

@petrochenkov
Copy link
Contributor

Hmm, Failed to clone 'src/llvm' a second time, aborting

@bors retry

@bors
Copy link
Contributor

bors commented Jan 2, 2017

⌛ Testing commit 3caf478 with merge 5d451cd...

@bors
Copy link
Contributor

bors commented Jan 2, 2017

💔 Test failed - status-travis

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Jan 4, 2017
…nkov

Add test for correct span for type

Adds test for and closes rust-lang#27522.
@estebank
Copy link
Contributor Author

estebank commented Jan 4, 2017

Had to change test as the error is pointing at the type now instead of the argument. Rebased and fixed. Can I get a new r+ for the rebase, @petrochenkov?

The error difference is:

  1 - |  --> $DIR/issue-27522.rs:16:16|
    + |  --> $DIR/issue-27522.rs:16:22|
  4 - |   |                ^^^^ expected Self, found struct `SomeType`|
    + |   |                      ^^^^^^^^^ expected Self, found struct `SomeType`|

@petrochenkov
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Jan 5, 2017

📌 Commit 1a4a6b9 has been approved by petrochenkov

@bors
Copy link
Contributor

bors commented Jan 9, 2017

⌛ Testing commit 1a4a6b9 with merge 88c6820...

@bors
Copy link
Contributor

bors commented Jan 9, 2017

💔 Test failed - status-travis

@estebank
Copy link
Contributor Author

estebank commented Jan 9, 2017

@petrochenkov can I get a retry? The travis error is unrelated to this PR:

 Downloading num_cpus v0.2.13
warning: spurious network error (2 tries remaining): [28] Timeout was reached (Operation too slow. Less than 10 bytes/sec transferred the last 30 seconds)
warning: spurious network error (1 tries remaining): [28] Timeout was reached (Operation too slow. Less than 10 bytes/sec transferred the last 30 seconds)
error: unable to get packages from source

@alexcrichton
Copy link
Member

alexcrichton commented Jan 9, 2017 via email

sanxiyn added a commit to sanxiyn/rust that referenced this pull request Jan 10, 2017
…nkov

Add test for correct span for type

Adds test for and closes rust-lang#27522.
bors added a commit that referenced this pull request Jan 10, 2017
Rollup of 11 pull requests

- Successful merges: #38606, #38607, #38623, #38664, #38799, #38816, #38836, #38839, #38841, #38849, #38874
- Failed merges: #38845
@bors bors merged commit 1a4a6b9 into rust-lang:master Jan 10, 2017
@estebank estebank deleted the test-for-27522 branch November 9, 2023 05:27
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.

Bad span for self type
6 participants