-
Notifications
You must be signed in to change notification settings - Fork 12.6k
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
Rename Receiver -> LegacyReceiver #130225
base: master
Are you sure you want to change the base?
Conversation
@rustbot label +S-waiting-on-author -S-waiting-on-review |
This comment has been minimized.
This comment has been minimized.
9a47840
to
827306a
Compare
This comment has been minimized.
This comment has been minimized.
827306a
to
7614901
Compare
This comment has been minimized.
This comment has been minimized.
7614901
to
9040bb1
Compare
This comment has been minimized.
This comment has been minimized.
9040bb1
to
dd2218c
Compare
@rustbot label -S-waiting-on-author +S-waiting-on-review |
☔ The latest upstream changes (presumably #130724) made this pull request unmergeable. Please resolve the merge conflicts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Compiler changes look good to me but I'm going to have T-libs sign off since this is a change to the standard library.
r? libs |
Looks like some name change was approved by the RFC, so I don't need to punt to a team, and I don't have strong feelings towards anything. This needs a rebase, but otherwise r=me and wesleywiser. @bors delegate+ |
✌️ @adetaylor, you can now approve this pull request! If @jhpratt told you to " |
There are merge commits (commits with multiple parents) in your changes. We have a no merge policy so these commits will need to be removed for this pull request to be merged. You can start a rebase with the following commands:
The following commits are merge commits: |
This comment has been minimized.
This comment has been minimized.
As part of the "arbitrary self types v2" project, we are going to replace the current `Receiver` trait with a new mechanism based on a new, different `Receiver` trait. This PR renames the old trait to get it out the way. Naming is hard. Options considered included: * HardCodedReceiver (because it should only be used for things in the standard library, and hence is sort-of hard coded) * LegacyReceiver * TargetLessReceiver * OldReceiver These are all bad names, but fortunately this will be temporary. Assuming the new mechanism proceeds to stabilization as intended, the legacy trait will be removed altogether. Although we expect this trait to be used only in the standard library, we suspect it may be in use elsehwere, so we're landing this change separately to identify any surprising breakages. It's known that this trait is used within the Rust for Linux project; a patch is in progress to remove their dependency. This is a part of the arbitrary self types v2 project, rust-lang/rfcs#3519 rust-lang#44874 r? @wesleywiser
fc5e6cc
to
20d50af
Compare
As part of the "arbitrary self types v2" project, we are going to replace the current
Receiver
trait with a new mechanism based on a new, differentReceiver
trait.This PR renames the old trait to get it out the way. Naming is hard. Options considered included:
These are all bad names, but fortunately this will be temporary. Assuming the new mechanism proceeds to stabilization as intended, the legacy trait will be removed altogether.
Although we expect this trait to be used only in the standard library, we suspect it may be in use elsehwere, so we're landing this change separately to identify any surprising breakages.
It's known that this trait is used within the Rust for Linux project; a patch is in progress to remove their dependency.
This is a part of the arbitrary self types v2 project,
rust-lang/rfcs#3519
#44874
r? @wesleywiser