Fix new clippy lints

canac · Nov 30, 2024 · e059244 · e059244
1 parent 43979bd
commit e059244
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/cli/src/import.rs b/cli/src/import.rs
@@ -59,7 +59,7 @@ where
 #[allow(clippy::module_name_repetitions)]
 pub async fn import_messages<B: Backend>(
     db: &Database<B>,
-    config: &Option<Config>,
+    config: Option<&Config>,
     new_messages: Vec<NewMessage>,
 ) -> Result<Vec<Message>> {
     let messages = db

diff --git a/cli/src/main.rs b/cli/src/main.rs
@@ -137,13 +137,17 @@ async fn run<B: Backend + Send + Sync + 'static>(
                 content,
                 state: Some(cli_state),
             }];
-            let messages = import_messages(&db, &config, raw_messages).await?;
+            let messages = import_messages(&db, config.as_ref(), raw_messages).await?;
             print!("{}", formatter.format_messages(&messages)?);
         }
 
         Command::Import { format } => {
-            let messages =
-                import_messages(&db, &config, read_messages_stdin(stdin().lock(), format)).await?;
+            let messages = import_messages(
+                &db,
+                config.as_ref(),
+                read_messages_stdin(stdin().lock(), format),
+            )
+            .await?;
             print!("{}", formatter.format_messages(&messages)?);
         }
 

diff --git a/database/src/filter.rs b/database/src/filter.rs
@@ -8,6 +8,7 @@ use std::str::FromStr;
 use std::string::ToString;
 
 // Serialize Option<Vec<T>> into a comma-separated string so that serde_urlencoded can handle it
+#[allow(clippy::ref_option)]
 fn serialize_vec_to_csv<S, T>(vec: &Option<Vec<T>>, serializer: S) -> Result<S::Ok, S::Error>
 where
     S: Serializer,