Skip to content

Remove a token after closing delimiter from the span of macro in type position #46835

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

Merged

Conversation

topecongiro
Copy link
Contributor

e.g.

let x = y: foo!();

The span for foo!() includes ;.

cc rust-lang/rustfmt#2290.

@rust-highfive
Copy link
Contributor

r? @petrochenkov

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

@topecongiro topecongiro force-pushed the bad-span-for-macro-invocation-in-type branch from 4fd04f9 to e0e62fc Compare December 19, 2017 16:22
@petrochenkov
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Dec 19, 2017

📌 Commit e0e62fc has been approved by petrochenkov

kennytm added a commit to kennytm/rust that referenced this pull request Dec 20, 2017
…cation-in-type, r=petrochenkov

Remove a token after closing delimiter from the span of macro in type position

e.g.
```rust
let x = y: foo!();
```
The span for `foo!()` includes `;`.

cc rust-lang/rustfmt#2290.
@kennytm kennytm added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Dec 20, 2017
bors added a commit that referenced this pull request Dec 20, 2017
Rollup of 14 pull requests

- Successful merges: #46359, #46517, #46671, #46751, #46760, #46787, #46794, #46828, #46831, #46835, #46851, #46852, #46856, #46870
- Failed merges:
@bors bors merged commit e0e62fc into rust-lang:master Dec 20, 2017
@topecongiro topecongiro deleted the bad-span-for-macro-invocation-in-type branch December 20, 2017 23:01
# 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