src: remove Clone from HttpClient impls

Author: Fishrock123

src/h1.rs

@@ -22,12 +22,6 @@ impl H1Client {
     }
 }
 
-impl Clone for H1Client {
-    fn clone(&self) -> Self {
-        Self {}
-    }
-}
-
 #[async_trait]
 impl HttpClient for H1Client {
     async fn send(&self, mut req: Request) -> Result<Response, Error> {

src/hyper.rs

@@ -11,7 +11,6 @@ use std::convert::TryFrom;
 use std::fmt::Debug;
 use std::io;
 use std::str::FromStr;
-use std::sync::Arc;
 
 type HyperRequest = hyper::Request<hyper::Body>;
 
@@ -28,28 +27,22 @@ impl<C: Clone + Connect + Debug + Send + Sync + 'static> HyperClientObject for h
 
 /// Hyper-based HTTP Client.
 #[derive(Debug)]
-pub struct HyperClient(Arc<dyn HyperClientObject>);
+pub struct HyperClient(Box<dyn HyperClientObject>);
 
 impl HyperClient {
     /// Create a new client instance.
     pub fn new() -> Self {
         let https = HttpsConnector::new();
         let client = hyper::Client::builder().build(https);
-        Self(Arc::new(client))
+        Self::from_client(client)
     }
 
     /// Create from externally initialized and configured client.
     pub fn from_client<C>(client: hyper::Client<C>) -> Self
     where
         C: Clone + Connect + Debug + Send + Sync + 'static,
     {
-        Self(Arc::new(client))
-    }
-}
-
-impl Clone for HyperClient {
-    fn clone(&self) -> Self {
-        Self(self.0.clone())
+        Self(Box::new(client))
     }
 }
 

src/isahc.rs

@@ -4,13 +4,10 @@ use super::{async_trait, Body, Error, HttpClient, Request, Response};
 
 use async_std::io::BufReader;
 use isahc::http;
-use std::sync::Arc;
 
 /// Curl-based HTTP Client.
 #[derive(Debug)]
-pub struct IsahcClient {
-    client: Arc<isahc::HttpClient>,
-}
+pub struct IsahcClient(isahc::HttpClient);
 
 impl Default for IsahcClient {
     fn default() -> Self {
@@ -26,17 +23,7 @@ impl IsahcClient {
 
     /// Create from externally initialized and configured client.
     pub fn from_client(client: isahc::HttpClient) -> Self {
-        Self {
-            client: Arc::new(client),
-        }
-    }
-}
-
-impl Clone for IsahcClient {
-    fn clone(&self) -> Self {
-        Self {
-            client: self.client.clone(),
-        }
+        Self(client)
     }
 }
 
@@ -59,7 +46,7 @@ impl HttpClient for IsahcClient {
         };
 
         let request = builder.body(body).unwrap();
-        let res = self.client.send_async(request).await.map_err(Error::from)?;
+        let res = self.0.send_async(request).await.map_err(Error::from)?;
         let (parts, body) = res.into_parts();
         let len = body.len().map(|len| len as usize);
         let body = Body::from_reader(BufReader::new(body), len);

src/wasm.rs

@@ -10,20 +10,12 @@ use std::task::{Context, Poll};
 
 /// WebAssembly HTTP Client.
 #[derive(Debug)]
-pub struct WasmClient {
-    _priv: (),
-}
+pub struct WasmClient {}
 
 impl WasmClient {
     /// Create a new instance.
     pub fn new() -> Self {
-        Self { _priv: () }
-    }
-}
-
-impl Clone for WasmClient {
-    fn clone(&self) -> Self {
-        Self { _priv: () }
+        Self {}
     }
 }