From 8206939fed0e2307455d46620ee114f74ab35d7f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lauren=C8=9Biu=20Nicola?= Date: Wed, 26 May 2021 18:34:50 +0300 Subject: clippy::redundant_clone fixes --- crates/hir_ty/src/infer/coerce.rs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'crates/hir_ty/src/infer/coerce.rs') diff --git a/crates/hir_ty/src/infer/coerce.rs b/crates/hir_ty/src/infer/coerce.rs index 765a02b1c..03b97e7db 100644 --- a/crates/hir_ty/src/infer/coerce.rs +++ b/crates/hir_ty/src/infer/coerce.rs @@ -76,17 +76,17 @@ impl<'a> InferenceContext<'a> { // way around first would mean we make the type variable `!`, instead of // just marking it as possibly diverging. if self.coerce(&ty2, &ty1) { - ty1.clone() + ty1 } else if self.coerce(&ty1, &ty2) { - ty2.clone() + ty2 } else { if let Some(id) = id { self.result .type_mismatches - .insert(id.into(), TypeMismatch { expected: ty1.clone(), actual: ty2.clone() }); + .insert(id.into(), TypeMismatch { expected: ty1.clone(), actual: ty2 }); } cov_mark::hit!(coerce_merge_fail_fallback); - ty1.clone() + ty1 } } @@ -183,7 +183,7 @@ impl<'a> InferenceContext<'a> { // details of coercion errors though, so I think it's useful to leave // the structure like it is. - let canonicalized = self.canonicalize(from_ty.clone()); + let canonicalized = self.canonicalize(from_ty); let autoderef = autoderef::autoderef( self.db, self.resolver.krate(), @@ -389,7 +389,7 @@ impl<'a> InferenceContext<'a> { // The CoerceUnsized trait should have two generic params: Self and T. return Err(TypeError); } - b.push(coerce_from.clone()).push(to_ty.clone()).build() + b.push(coerce_from).push(to_ty.clone()).build() }; let goal: InEnvironment = -- cgit v1.2.3