Skip to content

Commit f770921

Browse files
committed
Resolve assigning_clones clippy lint
warning: assigning the result of `Clone::clone()` may be inefficient --> impl/src/ast.rs:85:21 | 85 | *display = attrs.display.clone(); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `clone_from()`: `display.clone_from(&attrs.display)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assigning_clones = note: `#[warn(clippy::assigning_clones)]` on by default warning: assigning the result of `Clone::clone()` may be inefficient --> impl/src/expand.rs:158:9 | 158 | display_implied_bounds = display.implied_bounds.clone(); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `clone_from()`: `display_implied_bounds.clone_from(&display.implied_bounds)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assigning_clones warning: assigning the result of `Clone::clone()` may be inefficient --> impl/src/expand.rs:402:21 | 402 | display_implied_bounds = display.implied_bounds.clone(); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `clone_from()`: `display_implied_bounds.clone_from(&display.implied_bounds)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assigning_clones
1 parent 1d106b1 commit f770921

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

impl/src/ast.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ impl<'a> Enum<'a> {
8282
.map(|node| {
8383
let mut variant = Variant::from_syn(node, &scope, span)?;
8484
if let display @ None = &mut variant.attrs.display {
85-
*display = attrs.display.clone();
85+
display.clone_from(&attrs.display);
8686
}
8787
if let Some(display) = &mut variant.attrs.display {
8888
display.expand_shorthand(&variant.fields);

impl/src/expand.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ fn impl_struct(input: Struct) -> TokenStream {
155155
::core::fmt::Display::fmt(&self.#only_field, __formatter)
156156
})
157157
} else if let Some(display) = &input.attrs.display {
158-
display_implied_bounds = display.implied_bounds.clone();
158+
display_implied_bounds.clone_from(&display.implied_bounds);
159159
let use_as_display = use_as_display(display.has_bonus_display);
160160
let pat = fields_pat(&input.fields);
161161
Some(quote! {
@@ -399,7 +399,7 @@ fn impl_enum(input: Enum) -> TokenStream {
399399
let mut display_implied_bounds = Set::new();
400400
let display = match &variant.attrs.display {
401401
Some(display) => {
402-
display_implied_bounds = display.implied_bounds.clone();
402+
display_implied_bounds.clone_from(&display.implied_bounds);
403403
display.to_token_stream()
404404
}
405405
None => {

0 commit comments

Comments
 (0)