Skip to content

Remove cross-borrowing for traits#16917

Merged
bors merged 2 commits into
rust-lang:masterfrom
nrc:cross-trait
Sep 3, 2014
Merged

Remove cross-borrowing for traits#16917
bors merged 2 commits into
rust-lang:masterfrom
nrc:cross-trait

Conversation

@nrc

@nrc nrc commented Sep 1, 2014

Copy link
Copy Markdown
Member

Closes #15349

r? @pcwalton (or anyone else)

@nrc

nrc commented Sep 1, 2014

Copy link
Copy Markdown
Member Author

Note that the first commit is a different PR - #16915

Nick Cameron added 2 commits September 3, 2014 08:29
For example `let _x: &Trait = &*(box Foo as Box<Trait>);`. There was a bug where no cleanup would be scheduled by the deref.

No test because cleanup-auto-borrow-obj.rs is a test for this once we remove trait cross-borrowing (done on another branch).
Closes rust-lang#15349

[breaking-change]

Trait objects are no longer implicitly coerced from Box<T> to &T. You must make an explicit coercion using `&*`.
bors added a commit that referenced this pull request Sep 3, 2014
@bors bors closed this Sep 3, 2014
@bors bors merged commit 7f72884 into rust-lang:master Sep 3, 2014
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jun 11, 2026
)

Closes rust-lang/rust-clippy#16913

changelog: [`doc_markdown`] add common database engines to whitelist
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.

Remove cross-borrowing entirely

2 participants