From e2b0eb1fb99bcbaecd1793fdd3e17b03ea62c69f Mon Sep 17 00:00:00 2001 From: link2xt Date: Wed, 12 Aug 2026 19:20:53 +0000 Subject: [PATCH] chore: fix nightly clippy warnings --- src/blob.rs | 9 ++++----- src/imap.rs | 9 ++++----- src/securejoin/securejoin_tests.rs | 7 +++---- 3 files changed, 11 insertions(+), 14 deletions(-) diff --git a/src/blob.rs b/src/blob.rs index da5f8dfaa..97000849e 100644 --- a/src/blob.rs +++ b/src/blob.rs @@ -68,11 +68,10 @@ impl<'a> BlobObject<'a> { // so we are doing essentially the same here. task::block_in_place(|| { let temp_path; - let src_in_blobdir: &Path; let blobdir = context.get_blobdir(); - if src.starts_with(blobdir) { - src_in_blobdir = src; + let src_in_blobdir = if src.starts_with(blobdir) { + src } else { info!( context, @@ -84,8 +83,8 @@ impl<'a> BlobObject<'a> { std::fs::create_dir_all(blobdir).log_err(context).ok(); std::fs::copy(src, &temp_path).context("Copying new blobfile failed")?; }; - src_in_blobdir = &temp_path; - } + &temp_path + }; let hash = file_hash(src_in_blobdir)?.to_hex(); let hash = hash.as_str(); diff --git a/src/imap.rs b/src/imap.rs index 6378137b1..92a5ace5d 100644 --- a/src/imap.rs +++ b/src/imap.rs @@ -800,13 +800,12 @@ impl Session { context: &Context, folder: &str, ) -> Result<()> { - let uid_validity; // Collect pairs of UID and Message-ID. let mut msgs = BTreeMap::new(); let folder_exists = self.select_with_uidvalidity(context, folder).await?; let transport_id = self.transport_id(); - if folder_exists { + let uid_validity = if folder_exists { let mut list = self .uid_fetch("1:*", RFC724MID_UID) .await @@ -832,11 +831,11 @@ impl Session { msgs.len(), ); - uid_validity = get_uidvalidity(context, transport_id, folder).await?; + get_uidvalidity(context, transport_id, folder).await? } else { warn!(context, "resync_uids_with_server: No folder {folder}."); - uid_validity = 0; - } + 0 + }; // Write collected UIDs to SQLite database. context diff --git a/src/securejoin/securejoin_tests.rs b/src/securejoin/securejoin_tests.rs index 1b15b82f5..43ca95573 100644 --- a/src/securejoin/securejoin_tests.rs +++ b/src/securejoin/securejoin_tests.rs @@ -64,13 +64,12 @@ async fn test_setup_contact_ex(case: SetupContactCase) -> (TestContext, TestCont bob.set_config(Config::Displayname, Some("Bob Examplenet")) .await .unwrap(); - let alice_auto_submitted_hdr: bool; - match case { + let alice_auto_submitted_hdr = match case { SetupContactCase::AliceIsBot => { alice.set_config_bool(Config::Bot, true).await.unwrap(); - alice_auto_submitted_hdr = true; + true } - _ => alice_auto_submitted_hdr = false, + _ => false, }; assert_eq!(