feat: clippy clear

verdiwm · Feb 26, 2025 · 461295d · 461295d
1 parent e71603d
commit 461295d
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 9 deletions.
diff --git a/gen/src/client.rs b/gen/src/client.rs
@@ -29,7 +29,7 @@ pub fn generate_client_code(current: &[Pair], pairs: &[Pair]) -> TokenStream {
             let name = &interface.name;
             let version = &interface.version;
 
-            let enums = write_enums(&interface);
+            let enums = write_enums(interface);
 
             let requests = write_requests(pairs, pair, interface);
             let events = write_events(pairs, pair, interface);

diff --git a/gen/src/parser.rs b/gen/src/parser.rs
@@ -186,8 +186,7 @@ impl Arg {
                             pair.protocol
                                 .interfaces
                                 .iter()
-                                .find(|e| e.name == enum_interface)
-                                .is_some()
+                                .any(|e| e.name == enum_interface)
                         })
                         .unwrap()
                         .clone(),

diff --git a/gen/src/server.rs b/gen/src/server.rs
@@ -30,10 +30,10 @@ pub fn generate_server_code(current: &[Pair], pairs: &[Pair]) -> TokenStream {
             let name = &interface.name;
             let version = &interface.version;
 
-            let dispatchers = write_dispatchers(&interface);
-            let requests = write_requests(pairs, pair, &interface);
-            let events = write_events(pairs, pair, &interface);
-            let enums = write_enums(&interface);
+            let dispatchers = write_dispatchers(interface);
+            let requests = write_requests(pairs, pair, interface);
+            let events = write_events(pairs, pair, interface);
+            let enums = write_enums(interface);
 
             let handler_args = if dispatchers.is_empty() {
                 quote! {
@@ -104,7 +104,7 @@ fn write_dispatchers(interface: &Interface) -> Vec<TokenStream> {
 
     for (opcode, request) in interface.requests.iter().enumerate() {
         let opcode = opcode as u16;
-        let name = make_ident(&request.name.to_snek_case());
+        let name = make_ident(request.name.to_snek_case());
 
         let mut tracing_fmt = Vec::new();
         let mut tracing_args = Vec::new();
@@ -228,7 +228,7 @@ fn write_events(pairs: &[Pair], pair: &Pair, interface: &Interface) -> Vec<Token
         let mut tracing_args = Vec::new();
 
         for arg in &event.args {
-            let mut ty = arg.to_rust_type_token(arg.find_protocol(&pairs).as_ref().unwrap_or(pair));
+            let mut ty = arg.to_rust_type_token(arg.find_protocol(pairs).as_ref().unwrap_or(pair));
 
             let mut map_display = quote! {};
 

diff --git a/src/wire/message.rs b/src/wire/message.rs
@@ -3,6 +3,7 @@ use std::os::fd::RawFd;
 use bytes::{Buf, BufMut, Bytes, BytesMut};
 use rustix::fd::{FromRawFd, OwnedFd};
 
+#[cfg(feature = "server")]
 use crate::server::Client;
 
 use super::{DecodeError, Fixed, NewId, ObjectId};
@@ -155,6 +156,7 @@ impl Message {
             .ok_or(DecodeError::MalformedPayload)
     }
 
+    #[cfg(feature = "server")]
     pub async fn send(self, client: &mut Client) -> std::io::Result<()> {
         client.send_message(self).await
     }