Remove one unnecessary type.

src/network/client.rs

@@ -41,12 +41,7 @@ where
         task.run().await;
     });
 
-    (
-        RPCInterface { cmd_send },
-        ServerEventStream {
-            packets: Box::pin(event_receive),
-        },
-    )
+    (RPCInterface { cmd_send }, Box::pin(event_receive))
 }
 
 enum ClientTaskCommand<PacketType, ErrorType> {
@@ -306,21 +301,8 @@ impl<PacketType, ErrorType> Future for RPC<PacketType, ErrorType> {
     }
 }
 
-// TODO: Do we even need this type?
-pub struct ServerEventStream<PacketType> {
-    packets: Pin<Box<mpsc::Receiver<PacketType>>>,
-}
-
-impl<PacketType> Stream for ServerEventStream<PacketType> {
-    type Item = PacketType;
-
-    fn poll_next(self: Pin<&mut Self>, cx: &mut Context) -> Poll<Option<Self::Item>> {
-        // Safe, as we will not move self_.
-        let self_ = unsafe { self.get_unchecked_mut() };
-
-        Pin::as_mut(&mut self_.packets).poll_next(cx)
-    }
-}
+/// Stream of asynchronous packets received from the server.
+type ServerEventStream<PacketType> = Pin<Box<mpsc::Receiver<PacketType>>>;
 
 #[derive(Debug, PartialEq)]
 pub enum RPCError<StreamError> {