improve code by clippy

README.md

@@ -194,7 +194,7 @@ o2node-.->collector
 - [x] http_filter, websocket(including early data)
 - [x] h2, grpc
 - [x] quic
-- [x] ip relay protocol test1
+- [x] ip relay protocol (test1)
 - [ ] tcp/ip netstack (smoltcp)
 - [ ] ss?, vmess?
 

rucimp/src/net/http.rs

@@ -65,7 +65,7 @@ pub fn match_request_http_header<'a, T: 'a + std::fmt::Debug>(
     //debug!("checking  {r:?}");
 
     if c.authority != given_host {
-        if given_host == "" {
+        if given_host.is_empty() {
             let hh = r
                 .headers()
                 .get("host")

src/map/network/mod.rs

@@ -6,7 +6,6 @@ pub mod accept;
 pub mod echo;
 
 use macro_map::*;
-use parking_lot::Mutex;
 use tokio::sync::mpsc::Receiver;
 use tracing::debug;
 use tracing::info;
@@ -105,6 +104,7 @@ impl Map for Direct {
     }
 }
 
+#[cfg(feature = "tun")]
 #[derive(Clone, Debug, Default)]
 enum AutoRouteState {
     #[default]
@@ -127,7 +127,8 @@ pub struct BindDialer {
     #[cfg(feature = "tun")]
     pub out_auto_route: Option<tun::route::OutAutoRouteParams>,
 
-    auto_route_state: Arc<Mutex<AutoRouteState>>,
+    #[cfg(feature = "tun")]
+    auto_route_state: Arc<parking_lot::Mutex<AutoRouteState>>,
 }
 
 impl Name for BindDialer {
@@ -190,8 +191,8 @@ impl BindDialer {
 
         match r {
             Ok(c) => {
+                #[cfg(feature = "tun")]
                 if let Some(a) = &bind_a {
-                    #[cfg(feature = "tun")]
                     if let Network::IP = a.network {
                         if let Some(c) = &self.in_auto_route {
                             let mut mg = self.auto_route_state.lock();

src/map/stdio.rs

@@ -55,7 +55,7 @@ impl fmt::Display for HexSlice<'_> {
         for byte in self.0 {
             write!(f, "{:02X}", byte)?;
         }
-        write!(f, "\n")?;
+        writeln!(f)?;
         Ok(())
     }
 }

src/net/dns.rs

@@ -32,9 +32,10 @@ pub fn get_sys_dns() -> Vec<String> {
         }
     }
 
-    return vec![];
+    vec![]
 }
 
+#[allow(unused)]
 pub fn set_sys_dns(list: Vec<&str>) -> std::io::Result<()> {
     #[cfg(target_os = "linux")]
     {

src/net/tun/mod.rs

@@ -1,3 +1,4 @@
+#[allow(unused)]
 pub mod route;
 
 use anyhow::Context;
@@ -47,8 +48,6 @@ where
         "tun: create_bind succeed"
     );
 
-    // let dev = crate::net::helpers::PrintWrapper::from(Box::new(dev));
-
     Ok(Box::new(dev))
 }
 

src/net/tun/route.rs

@@ -27,6 +27,7 @@ pub struct OutAutoRouteParams {
 const DEFAULT_ROUTER_IP: &str = "192.168.0.1";
 const DEFAULT_ORIGINAL_DEV_NAME: &str = "enp0s1";
 
+#[allow(unused)]
 pub fn out_auto_route(params: &OutAutoRouteParams) -> anyhow::Result<()> {
     #[cfg(target_os = "linux")]
     {
@@ -60,6 +61,7 @@ iptables -t nat -I POSTROUTING -o {original_dev_name} -j MASQUERADE"#,
     Ok(())
 }
 
+#[allow(unused)]
 pub fn out_down_route(params: &OutAutoRouteParams) -> anyhow::Result<()> {
     #[cfg(target_os = "linux")]
     {