Merge pull request #277 from espindola/pin

Drop unnecessary use of Pin

Author: blackbeam

src/conn/pool/futures/get_conn.rs

@@ -113,7 +113,7 @@ impl Future for GetConn {
             match self.inner {
                 GetConnInner::New => {
                     let queue_id = self.queue_id;
-                    let next = ready!(Pin::new(self.pool_mut()).poll_new_conn(cx, queue_id))?;
+                    let next = ready!(self.pool_mut().poll_new_conn(cx, queue_id))?;
                     match next {
                         GetConnInner::Connecting(conn_fut) => {
                             self.inner = GetConnInner::Connecting(conn_fut);

src/conn/pool/mod.rs

@@ -16,7 +16,6 @@ use std::{
     collections::VecDeque,
     convert::TryFrom,
     hash::{Hash, Hasher},
-    pin::Pin,
     str::FromStr,
     sync::{atomic, Arc, Mutex},
     task::{Context, Poll, Waker},
@@ -286,7 +285,7 @@ impl Pool {
 
     /// Poll the pool for an available connection.
     fn poll_new_conn(
-        self: Pin<&mut Self>,
+        &mut self,
         cx: &mut Context<'_>,
         queue_id: QueueId,
     ) -> Poll<Result<GetConnInner>> {