Skip to content

Add #![feature(ergonomic_clones)] formatting#6532

Merged
ytmimi merged 1 commit intorust-lang:masterfrom
ytmimi:ergonomic_clones_formatting
Apr 2, 2025
Merged

Add #![feature(ergonomic_clones)] formatting#6532
ytmimi merged 1 commit intorust-lang:masterfrom
ytmimi:ergonomic_clones_formatting

Conversation

@ytmimi
Copy link
Contributor

@ytmimi ytmimi commented Apr 2, 2025

Tracking issue rust-lang/rust#134797

Closes #6512

Tracking issue r-l/rust#134797

Fixes 6512
@ytmimi ytmimi force-pushed the ergonomic_clones_formatting branch from 7391e74 to 7265718 Compare April 2, 2025 15:02
@ytmimi
Copy link
Contributor Author

ytmimi commented Apr 2, 2025

@spastorino thanks for taking a look at the implementation. I'll go ahead and get this merged now and the formatting will be available in r-l/rust after the next subtree sync, which I hope to get out soon(ish).

@ytmimi ytmimi merged commit cdccf23 into rust-lang:master Apr 2, 2025
30 checks passed
@ytmimi ytmimi deleted the ergonomic_clones_formatting branch April 2, 2025 15:32
@ytmimi ytmimi added release-notes Needs an associated changelog entry and removed pr-not-reviewed labels Apr 2, 2025
@spastorino
Copy link
Member

@spastorino thanks for taking a look at the implementation. I'll go ahead and get this merged now and the formatting will be available in r-l/rust after the next subtree sync, which I hope to get out soon(ish).

I've opened this rust-lang/rust#139692, not sure when this change is going to be used in rust-lang/rust's rustfmt. Please let me know when the sync happens so I can rebase the PR and mark it as ready.

@spastorino
Copy link
Member

@ytmimi I didn't do a proper check to be honest, but I was surprised that this is not yet included in rustfmt 1.8.0-nightly (ce7e97f737 2025-05-11).

@ytmimi ytmimi removed the release-notes Needs an associated changelog entry label Feb 26, 2026
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.

Add ergonomic ref counting support

3 participants