Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix span for type-only arguments #50979

Merged
merged 1 commit into from
May 24, 2018
Merged

Conversation

Manishearth
Copy link
Member

Currently it points to the comma or parenthesis before the type, which is broken

cc @mark-i-m this is what broke #48309

r? @estebank

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 22, 2018
@mark-i-m
Copy link
Member

Thanks @Manishearth! Should I change the other PR back to using arg.pat.span?

@estebank
Copy link
Contributor

@bors r+ rollup

I'll let #48309 introduce the test for this.

@mark-i-m I think that would be best.

@bors
Copy link
Contributor

bors commented May 22, 2018

📌 Commit d7086ca has been approved by estebank

@bors bors 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 May 22, 2018
@Manishearth
Copy link
Member Author

Well, both are correct.

But yeah, with pat.span this lets that PR introduce a regression test.

@mark-i-m
Copy link
Member

Done. Thanks!

kennytm added a commit to kennytm/rust that referenced this pull request May 24, 2018
Fix span for type-only arguments

Currently it points to the comma or parenthesis before the type, which is broken

cc @mark-i-m this is what broke rust-lang#48309

r? @estebank
bors added a commit that referenced this pull request May 24, 2018
Rollup of 9 pull requests

Successful merges:

 - #50864 (Add NetBSD/arm target specs)
 - #50956 (rust-gdb: work around the re-used -d argument in cgdb)
 - #50964 (Make sure that queries have predictable symbol names.)
 - #50965 (Update LLVM to pull in another wasm fix)
 - #50972 (Add -Z no-parallel-llvm flag)
 - #50979 (Fix span for type-only arguments)
 - #50981 (Shrink `LiveNode`.)
 - #50995 (move type out of unsafe block)
 - #51011 ( rustdoc: hide macro export statements from docs)

Failed merges:
@bors bors merged commit d7086ca into rust-lang:master May 24, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
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