Skip to content

Explicitly impl Clone for RWArc - #8198

Closed
lilyball wants to merge 1 commit into
rust-lang:masterfrom
lilyball:clone-rwarc
Closed

Explicitly impl Clone for RWArc#8198
lilyball wants to merge 1 commit into
rust-lang:masterfrom
lilyball:clone-rwarc

Conversation

@lilyball

Copy link
Copy Markdown
Contributor

RWArc had a clone() method, but it was part of impl RWArc instead of
an implementation of Clone.

Stick with the explicit implementation instead of deriving Clone so we
can have a docstring.

Fixes#8052.

RWArc had a clone() method, but it was part of impl RWArc instead of
an implementation of Clone.
Stick with the explicit implementation instead of deriving Clone so we
can have a docstring.
Fixesrust-lang#8052.
@lilyball

Copy link
Copy Markdown
ContributorAuthor

r? @bblum

I standardized the docstring on all 3 clone()s to be on the method.

@thestinger

Copy link
Copy Markdown
Contributor

Landing with #8257.

flip1995 pushed a commit to flip1995/rust that referenced this pull request Jan 13, 2022
…p1995
Remove method_call! macro
This is possible now that `SymbolStr` is removed from rustc.
changelog: none
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
8200: Use arrayvec 0.6 r=kjeremy a=lnicola
Closesrust-lang#8198
changelog skip
Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>
Sign up for freeto 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.

RWARC should explicitly 'impl... Clone`

3 participants

@lilyball@thestinger@bblum