fix clippy warnings

stalwartlabs · Dec 6, 2023 · 7821347 · 7821347
1 parent e0030a2
commit 7821347
Show file tree

Hide file tree

Showing 3 changed files with 29 additions and 30 deletions.
diff --git a/src/decoders/quoted_printable.rs b/src/decoders/quoted_printable.rs
@@ -312,6 +312,27 @@ impl<'x> MessageStream<'x> {
         None
     }
 }
+
+/*
+ * Adapted from Daniel Lemire's source:
+ * https://github.com/lemire/Code-used-on-Daniel-Lemire-s-blog/blob/master/2019/04/17/hexparse.cpp
+ *
+ */
+
+pub static HEX_MAP: &[i8] = &[
+    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
+    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
+    0, 1, 2, 3, 4, 5, 6, 7, 8, 9, -1, -1, -1, -1, -1, -1, -1, 10, 11, 12, 13, 14, 15, -1, -1, -1,
+    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, 10,
+    11, 12, 13, 14, 15, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
+    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
+    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
+    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
+    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
+    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
+    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
+];
+
 #[cfg(test)]
 mod tests {
     use crate::parsers::MessageStream;
@@ -468,23 +489,3 @@ mod tests {
         }
     }
 }
-
-/*
- * Adapted from Daniel Lemire's source:
- * https://github.com/lemire/Code-used-on-Daniel-Lemire-s-blog/blob/master/2019/04/17/hexparse.cpp
- *
- */
-
-pub static HEX_MAP: &[i8] = &[
-    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
-    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
-    0, 1, 2, 3, 4, 5, 6, 7, 8, 9, -1, -1, -1, -1, -1, -1, -1, 10, 11, 12, 13, 14, 15, -1, -1, -1,
-    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, 10,
-    11, 12, 13, 14, 15, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
-    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
-    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
-    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
-    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
-    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
-    -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
-];
diff --git a/src/parsers/message.rs b/src/parsers/message.rs
@@ -314,7 +314,7 @@ impl MessageParser {
                     message.text_body.push(message.parts.len());
                 }
 
-                if !is_multipart && !(add_to_text || add_to_html)
+                if !is_multipart && !add_to_text && !add_to_html
                     || (is_media || is_multipart) && (!add_to_text || !add_to_html)
                 {
                     message.attachments.push(message.parts.len());
@@ -343,12 +343,10 @@ impl MessageParser {
                     } else {
                         PartType::Text(text)
                     }
+                } else if !is_inline {
+                    PartType::Binary(bytes)
                 } else {
-                    if !is_inline {
-                        PartType::Binary(bytes)
-                    } else {
-                        PartType::InlineBinary(bytes)
-                    }
+                    PartType::InlineBinary(bytes)
                 }
             } else {
                 message.attachments.push(message.parts.len());

diff --git a/src/parsers/mime.rs b/src/parsers/mime.rs
@@ -134,16 +134,16 @@ impl<'x> MessageStream<'x> {
         self.checkpoint();
 
         match (self.next(), self.peek()) {
-            (Some(&a), Some(&&b)) if a == b'\r' && b == b'\n' => {
+            (Some(b'\r'), Some(b'\n')) => {
                 self.next();
                 false
             }
-            (Some(&a), Some(&&b)) if a == b'-' && b == b'-' => {
+            (Some(b'-'), Some(b'-')) => {
                 self.next();
                 true
             }
-            (Some(&a), _) if a == b'\n' => false,
-            (Some(&a), _) if a.is_ascii_whitespace() => {
+            (Some(b'\n'), _) => false,
+            (Some(a), _) if a.is_ascii_whitespace() => {
                 self.skip_crlf();
                 false
             }