Avoid calling .map with function returning the unit type

It seems clearer to write this with "if let".

Found by clippy lint option_map_unit_fn:
https://rust-lang.github.io/rust-clippy/master/index.html#option_map_unit_fn

1 files changed, 3 insertions, 1 deletions

diff --git a/openpgp/src/parse.rs b/openpgp/src/parse.rs
index 5cf7c5f5..11f2f3e3 100644
--- a/openpgp/src/parse.rs
+++ b/openpgp/src/parse.rs
@@ -4853,7 +4853,9 @@ impl <'a> PacketParser<'a> {
     fn hash_read_content(&mut self, b: &[u8]) {
         if !b.is_empty() {
             assert!(self.body_hash.is_some());
-            self.body_hash.as_mut().map(|h| h.update(b));
+            if let Some(h) = self.body_hash.as_mut() {
+                h.update(b);
+            }
             self.content_was_read = true;
         }
     }