clone as little as possible into closures

Author: rklaehn

src/net_protocol.rs

@@ -393,24 +393,20 @@ impl<S: crate::store::Store> Blobs<S> {
 
 impl<S: crate::store::Store> ProtocolHandler for Blobs<S> {
     fn accept(&self, conn: Connecting) -> BoxedFuture<Result<()>> {
-        let this = self.clone();
+        let db = self.store.clone();
+        let events = self.events.clone();
+        let rt = self.rt.clone();
 
         Box::pin(async move {
-            crate::provider::handle_connection(
-                conn.await?,
-                this.store.clone(),
-                this.events.clone(),
-                this.rt.clone(),
-            )
-            .await;
+            crate::provider::handle_connection(conn.await?, db, events, rt).await;
             Ok(())
         })
     }
 
     fn shutdown(&self) -> BoxedFuture<()> {
-        let this = self.clone();
+        let store = self.store.clone();
         Box::pin(async move {
-            this.store.shutdown().await;
+            store.shutdown().await;
         })
     }
 }

src/rpc.rs

@@ -80,11 +80,9 @@ impl<D: crate::store::Store> Blobs<D> {
     where
         C: ChannelTypes<RpcService>,
     {
-        use Request::*;
-        let handler = self;
         match msg {
-            Blobs(msg) => handler.handle_blobs_request(msg, chan).await,
-            Tags(msg) => handler.handle_tags_request(msg, chan).await,
+            Request::Blobs(msg) => self.handle_blobs_request(msg, chan).await,
+            Request::Tags(msg) => self.handle_tags_request(msg, chan).await,
         }
     }