From c7b2bdfaac4416af9059a4e705231460d122af7d Mon Sep 17 00:00:00 2001
From: Alexander Krotov <ilabdsf@gmail.com>
Date: Sat, 11 Jan 2020 15:43:45 +0300
Subject: [PATCH] mimeparser: simplify get_subject() and get_rfc724_mid

---
 src/mimeparser.rs | 18 +++++-------------
 1 file changed, 5 insertions(+), 13 deletions(-)

diff --git a/src/mimeparser.rs b/src/mimeparser.rs
index a84a07410..95bb73127 100644
--- a/src/mimeparser.rs
+++ b/src/mimeparser.rs
@@ -367,14 +367,9 @@ impl<'a> MimeMessage<'a> {
     }
 
     pub(crate) fn get_subject(&self) -> Option<String> {
-        if let Some(s) = self.get(HeaderDef::Subject) {
-            if s.is_empty() {
-                return None;
-            }
-            Some(s.to_string())
-        } else {
-            None
-        }
+        self.get(HeaderDef::Subject)
+            .filter(|s| !s.is_empty())
+            .map(|s| s.to_string())
     }
 
     pub fn get(&self, headerdef: HeaderDef) -> Option<&String> {
@@ -713,11 +708,8 @@ impl<'a> MimeMessage<'a> {
     }
 
     pub fn get_rfc724_mid(&self) -> Option<String> {
-        if let Some(msgid) = self.get(HeaderDef::MessageId) {
-            parse_message_id(msgid)
-        } else {
-            None
-        }
+        self.get(HeaderDef::MessageId)
+            .and_then(|msgid| parse_message_id(msgid))
     }
 
     fn merge_headers(&mut self, fields: &[mailparse::MailHeader<'_>]) {