2

Add new ThinBox type for 1 stack pointer wide heap allocated trait objects by ya...

 2 years ago
source link: https://github.com/rust-lang/rust/pull/90066
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.

Conversation

Copy link

Collaborator

rust-highfive commented on Oct 19, 2021

r? @kennytm

(rust-highfive has picked a reviewer for you, use r? to override)

rust-highfive

added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label

on Oct 19, 2021

Copy link

Member

Author

yaahc commented on Oct 19, 2021

m-ou-se

added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.

S-experimental Status: Ongoing experiment that does not require reviewing and won't be merged in its current state.

and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.

labels

on Oct 19, 2021

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

Copy link

Contributor

bors commented on Oct 20, 2021

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

bors

added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label

on Oct 20, 2021

This comment has been minimized.

This comment has been minimized.

rustbot

added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label

on Jan 10

This comment has been minimized.

Copy link

Member

Author

yaahc commented on Feb 18

r? rust-lang/libs

yaahc

added T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.

and removed T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

labels

on Feb 18

Copy link

Member

Author

yaahc commented on Feb 18

second attempt!

r? rust-lang/libs

JohnCSimon

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

28 days ago

Copy link

Contributor

bors commented 16 days ago

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

@@ -152,6 +153,7 @@

#![feature(fundamental)]

#![cfg_attr(not(test), feature(generator_trait))]

#![feature(lang_items)]

#![feature(let_else)]

Copy link

Member

joshtriplett commented 10 days ago

Posted a few comments, and un-resolved one other comment that didn't get addressed.

LGTM otherwise; r=me with those addressed.

This comment has been minimized.

This comment has been minimized.

Copy link

Contributor

bors commented 9 days ago

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

Copy link

Member

Author

yaahc commented 9 days ago

@bors r=joshtriplett

Copy link

Contributor

bors commented 9 days ago

pushpin Commit a87a0d0 has been approved by joshtriplett

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

9 days ago

bors

merged commit ee8cea8 into

rust-lang:master 8 days ago

10 checks passed

rustbot

added this to the 1.62.0 milestone

8 days ago

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment


About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK