Skip to content
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 traits::Vtable to ImplSource. #72114

Merged
merged 2 commits into from
Jun 9, 2020
Merged

Conversation

anyska
Copy link
Contributor

@anyska anyska commented May 11, 2020

Originally suggested by @eddyb.

r? @nikomatsakis

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 11, 2020
@nikomatsakis
Copy link
Contributor

I agree that the name Vtable is confusing for folks. Not sure about Satisfier yet, I guess I have to ponder.

@nikomatsakis
Copy link
Contributor

@eddyb and I mentioned ImplSource in privmsg as an alternative

@Elinvynia Elinvynia added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 20, 2020
@bors
Copy link
Contributor

bors commented May 21, 2020

☔ The latest upstream changes (presumably #70705) made this pull request unmergeable. Please resolve the merge conflicts.

@nikomatsakis
Copy link
Contributor

@anyska how about renaming to ImplSource? (I still think that sounds better, do you agree @eddyb?)

@bors
Copy link
Contributor

bors commented May 27, 2020

☔ The latest upstream changes (presumably #71996) made this pull request unmergeable. Please resolve the merge conflicts.

@anyska anyska changed the title Rename traits::Vtable to Satisfier. Rename traits::Vtable to ImplSource. Jun 2, 2020
@bors
Copy link
Contributor

bors commented Jun 3, 2020

☔ The latest upstream changes (presumably #72948) made this pull request unmergeable. Please resolve the merge conflicts.

@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Jun 8, 2020

📌 Commit 00c19ad has been approved by nikomatsakis

@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 Jun 8, 2020
@bors
Copy link
Contributor

bors commented Jun 9, 2020

⌛ Testing commit 00c19ad with merge feb3536...

@bors
Copy link
Contributor

bors commented Jun 9, 2020

☀️ Test successful - checks-azure
Approved by: nikomatsakis
Pushing feb3536 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Jun 9, 2020
@bors bors mentioned this pull request Jun 9, 2020
@bors bors merged commit feb3536 into rust-lang:master Jun 9, 2020
ptrojahn added a commit to ptrojahn/rust that referenced this pull request Jul 9, 2021
As rust-lang#72114 and rust-lang#73055 were merged so closely together I think this
accidentally happened while rebasing
JohnTitor added a commit to JohnTitor/rust that referenced this pull request Jul 10, 2021
…-schievink

Revert the revert of renaming traits::VTable to ImplSource

As rust-lang#72114 and rust-lang#73055 were merged so closely together I think this
accidentally happened while rebasing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. 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