reference icon indicating copy to clipboard operation
reference copied to clipboard

Add trait_upcasting related languages changes

Open crlf0710 opened this issue 1 year ago • 6 comments

cc https://github.com/rust-lang/rust/issues/65991

crlf0710 avatar Sep 02 '22 17:09 crlf0710

What is the status here? The feature is being stabilized so maybe we should merge the reference changes :)

WaffleLapkin avatar Nov 22 '23 13:11 WaffleLapkin

What is the status here? The feature is being stabilized so maybe we should merge the reference changes :)

The reference changes have to wait for the stabilization PR to merge.

Sorry I didn't get to reviewing this earlier, I did not know it was being stabilized, and I tend to not prioritize pre-stabilization PRs when it seems like they are a long way off from being stabilized.

ehuss avatar Nov 24 '23 21:11 ehuss

Ping @crlf0710 Do you think you'd be able to address the comments here?

ehuss avatar Jan 22 '24 15:01 ehuss

This feature is being reverted so we should not land this PR.

That said, it'd still be good to have the PR ready for when this gets stabilized for real. :)

RalfJung avatar Jan 22 '24 15:01 RalfJung

Yea, I wanted to get this ready for when it lands again. The stabilization should not have been done without this being ready, but I let it fall through the cracks.

ehuss avatar Jan 22 '24 16:01 ehuss

Do you think you'd be able to address the comments here?

Frankly, i don't feel i have enough skill of writing to address the comments here(especially as i'm not a English-speaker). I can deal with "add XXX after YYY"-style of comments well, but i can't deal with the "This should mention" and "worth stating explicitly" style of comments.

crlf0710 avatar Jan 24 '24 03:01 crlf0710