Merge #1493

1493: Clippy trivially_copy_pass_by_ref r=matklad a=kjeremy

Clippy says that this is more efficient.

Co-authored-by: Jeremy Kolb <[email protected]>

1 files changed, 4 insertions, 4 deletions

diff --git a/crates/ra_hir/src/adt.rs b/crates/ra_hir/src/adt.rs
index 9c0c8fdd0..5a3ea5f55 100644
--- a/crates/ra_hir/src/adt.rs
+++ b/crates/ra_hir/src/adt.rs
@@ -34,8 +34,8 @@ impl AdtDef {
 }
 
 impl Struct {
-    pub(crate) fn variant_data(&self, db: &impl DefDatabase) -> Arc<VariantData> {
-        db.struct_data(*self).variant_data.clone()
+    pub(crate) fn variant_data(self, db: &impl DefDatabase) -> Arc<VariantData> {
+        db.struct_data(self).variant_data.clone()
     }
 }
 
@@ -69,7 +69,7 @@ fn variants(enum_def: &ast::EnumDef) -> impl Iterator<Item = &ast::EnumVariant>
 
 impl EnumVariant {
     pub(crate) fn source_impl(
-        &self,
+        self,
         db: &(impl DefDatabase + AstDatabase),
     ) -> Source<TreeArc<ast::EnumVariant>> {
         let src = self.parent.source(db);
@@ -81,7 +81,7 @@ impl EnumVariant {
             .to_owned();
         Source { file_id: src.file_id, ast }
     }
-    pub(crate) fn variant_data(&self, db: &impl DefDatabase) -> Arc<VariantData> {
+    pub(crate) fn variant_data(self, db: &impl DefDatabase) -> Arc<VariantData> {
         db.enum_data(self.parent).variants[self.id].variant_data.clone()
     }
 }