Remove custom NeverOkResult trait and instead use new Rust 1.82 matching

Author: faern

src/forward_traffic.rs

@@ -1,4 +1,3 @@
-use crate::NeverOkResult;
 use err_context::BoxedErrorExt as _;
 use err_context::ResultExt as _;
 use futures::future::select;
@@ -41,7 +40,7 @@ pub async fn process_udp_over_tcp(
         }
     };
     let udp2tcp = async move {
-        let error = process_udp2tcp(udp_in, tcp_out).await.into_error();
+        let Err(error) = process_udp2tcp(udp_in, tcp_out).await;
         log::error!("Error: {}", error.display("\nCaused by: "));
     };
 

src/lib.rs

@@ -95,15 +95,3 @@ pub use tcp_options::{ApplyTcpOptionsError, ApplyTcpOptionsErrorKind, TcpOptions
 
 /// Size of the header (in bytes) that is prepended to each datagram in the TCP stream.
 pub use forward_traffic::HEADER_LEN;
-
-/// Helper trait for `Result<Infallible, E>` types. Allows getting the `E` value
-/// in a way that is guaranteed to not panic.
-pub trait NeverOkResult<E> {
-    fn into_error(self) -> E;
-}
-
-impl<E> NeverOkResult<E> for Result<std::convert::Infallible, E> {
-    fn into_error(self) -> E {
-        self.expect_err("Result<Infallible, _> can't be Ok variant")
-    }
-}