Skip to content

Use constraint span when lowering associated types - #63620

Merged
bors merged 2 commits into
rust-lang:masterfrom
estebank:assoc-type-span
Aug 20, 2019
Merged

Use constraint span when lowering associated types#63620
bors merged 2 commits into
rust-lang:masterfrom
estebank:assoc-type-span

Conversation

@estebank

Copy link
Copy Markdown
Contributor

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 15, 2019
@rust-highfive

This comment has been minimized.

@estebank

Copy link
Copy Markdown
ContributorAuthor

cc @alexreg

@rust-highfive

This comment has been minimized.

@CentrilCentril added the F-associated_type_bounds `#![feature(associated_type_bounds)]` label Aug 16, 2019

@CentrilCentril left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heh; didn't expect anyone to fix this before I had filed my tests (including the one added here) as a PR. =P

Comment threadsrc/test/ui/associated-item/associated-item-type-issue-63594.rs Outdated
Comment threadsrc/librustc/hir/lowering.rs Outdated
@Centril

Copy link
Copy Markdown
Contributor

cc #52662

@Centril

Copy link
Copy Markdown
Contributor

r=me with changes ^---

@alexreg

Copy link
Copy Markdown
Contributor

Thanks @estebank!

@CentrilCentril added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 18, 2019
@estebank

Copy link
Copy Markdown
ContributorAuthor

@bors r=Centril

@bors

bors commented Aug 19, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 1808e4d has been approved by Centril

@borsbors 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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 19, 2019
Centril added a commit to Centril/rust that referenced this pull request Aug 19, 2019
@CentrilCentril mentioned this pull request Aug 19, 2019
bors added a commit that referenced this pull request Aug 19, 2019
Rollup of 5 pull requests
Successful merges:
- #63252 (Remove recommendation about idiomatic syntax for Arc::clone)
- #63376 (use different lifetime name for object-lifetime-default elision)
- #63620 (Use constraint span when lowering associated types)
- #63699 (Fix suggestion from incorrect `move async` to `async move`.)
- #63704 ( Fixed: error: unnecessary trailing semicolon)
Failed merges:
r? @ghost
@bors
bors merged commit 1808e4d into rust-lang:masterAug 20, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

F-associated_type_bounds`#![feature(associated_type_bounds)]`S-waiting-on-borsStatus: 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.

Incorrectly-/Un-spanned error with type Out = Box<dyn Bar<Assoc: Copy>>;

5 participants

@estebank@rust-highfive@Centril@alexreg@bors