From 849cde9757101104bb9d6c48473955aab069955b Mon Sep 17 00:00:00 2001
From: link2xt <link2xt@testrun.org>
Date: Wed, 19 Nov 2025 11:59:24 +0000
Subject: [PATCH] refactor: remove some easy to remove unwrap() calls

---
 src/configure.rs  | 2 +-
 src/mimeparser.rs | 5 +++--
 src/scheduler.rs  | 6 +++---
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/configure.rs b/src/configure.rs
index 1d0bc5c54..5b76b0920 100644
--- a/src/configure.rs
+++ b/src/configure.rs
@@ -525,7 +525,7 @@ async fn configure(ctx: &Context, param: &EnteredLoginParam) -> Result<Option<&'
     progress!(ctx, 850);
 
     // Wait for SMTP configuration
-    smtp_config_task.await.unwrap()?;
+    smtp_config_task.await??;
 
     progress!(ctx, 900);
 
diff --git a/src/mimeparser.rs b/src/mimeparser.rs
index 2301d3c2f..69bb19855 100644
--- a/src/mimeparser.rs
+++ b/src/mimeparser.rs
@@ -679,8 +679,9 @@ impl MimeMessage {
     fn parse_system_message_headers(&mut self, context: &Context) {
         if self.get_header(HeaderDef::AutocryptSetupMessage).is_some() && !self.incoming {
             self.parts.retain(|part| {
-                part.mimetype.is_none()
-                    || part.mimetype.as_ref().unwrap().as_ref() == MIME_AC_SETUP_FILE
+                part.mimetype
+                    .as_ref()
+                    .is_none_or(|mimetype| mimetype.as_ref() == MIME_AC_SETUP_FILE)
             });
 
             if self.parts.len() == 1 {
diff --git a/src/scheduler.rs b/src/scheduler.rs
index 5025d5023..97f90e85c 100644
--- a/src/scheduler.rs
+++ b/src/scheduler.rs
@@ -145,14 +145,14 @@ impl SchedulerState {
                 InnerSchedulerState::Started(_) => {
                     let new_state = InnerSchedulerState::Paused {
                         started: true,
-                        pause_guards_count: NonZeroUsize::new(1).unwrap(),
+                        pause_guards_count: NonZeroUsize::MIN,
                     };
                     Self::do_stop(&mut inner, context, new_state).await;
                 }
                 InnerSchedulerState::Stopped => {
                     *inner = InnerSchedulerState::Paused {
                         started: false,
-                        pause_guards_count: NonZeroUsize::new(1).unwrap(),
+                        pause_guards_count: NonZeroUsize::MIN,
                     };
                 }
                 InnerSchedulerState::Paused {
@@ -183,7 +183,7 @@ impl SchedulerState {
                     ref started,
                     ref mut pause_guards_count,
                 } => {
-                    if *pause_guards_count == NonZeroUsize::new(1).unwrap() {
+                    if *pause_guards_count == NonZeroUsize::MIN {
                         match *started {
                             true => SchedulerState::do_start(&mut inner, &context).await,
                             false => *inner = InnerSchedulerState::Stopped,