Drop unused return values of TlsListener::configure and TlsListener::connect

These methods return the TlsAcceptor and TcpListener, respectively,
but the only caller ignores those return values.

Author: joshtriplett

src/tls_listener.rs

@@ -69,7 +69,7 @@ impl<State> TlsListener<State> {
         TlsListenerBuilder::new()
     }
 
-    async fn configure(&mut self) -> io::Result<TlsAcceptor> {
+    async fn configure(&mut self) -> io::Result<()> {
         self.config = match std::mem::take(&mut self.config) {
             TlsListenerConfig::Paths { cert, key } => {
                 let certs = load_certs(&cert)?;
@@ -89,8 +89,8 @@ impl<State> TlsListener<State> {
             other => other,
         };
 
-        if let TlsListenerConfig::Acceptor(ref a) = self.config {
-            Ok(a.clone())
+        if let TlsListenerConfig::Acceptor(_) = self.config {
+            Ok(())
         } else {
             Err(io::Error::new(
                 io::ErrorKind::Other,
@@ -113,17 +113,12 @@ impl<State> TlsListener<State> {
         }
     }
 
-    async fn connect(&mut self) -> io::Result<&TcpListener> {
+    async fn connect(&mut self) -> io::Result<()> {
         if let TcpConnection::Addrs(addrs) = &self.connection {
             let tcp = TcpListener::bind(&addrs[..]).await?;
             self.connection = TcpConnection::Connected(tcp);
         }
-
-        if let TcpConnection::Connected(tcp) = &self.connection {
-            Ok(tcp)
-        } else {
-            unreachable!()
-        }
+        Ok(())
     }
 }