From e431ba0bf531dd6b52703b99b7ed3a7675cb61b3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Damir=20Jeli=C4=87?= <poljar@termina.org.uk>
Date: Mon, 3 Aug 2020 15:05:19 +0200
Subject: [PATCH] crypto: Fix some clippy warnings.

---
 matrix_sdk_crypto/src/verification/machine.rs | 9 ++++-----
 matrix_sdk_crypto/src/verification/sas/mod.rs | 6 ++----
 2 files changed, 6 insertions(+), 9 deletions(-)

diff --git a/matrix_sdk_crypto/src/verification/machine.rs b/matrix_sdk_crypto/src/verification/machine.rs
index d58c9dc4..579e0b2a 100644
--- a/matrix_sdk_crypto/src/verification/machine.rs
+++ b/matrix_sdk_crypto/src/verification/machine.rs
@@ -138,11 +138,10 @@ impl VerificationMachine {
             AnyToDeviceEvent::KeyVerificationMac(e) => {
                 if let Some(s) = self.get_sas(&e.content.transaction_id) {
                     self.receive_event_helper(&s, event);
-                    if s.is_done() {
-                        if !s.mark_device_as_verified().await? {
-                            if let Some(r) = s.cancel() {
-                                self.outgoing_to_device_messages.insert(r.txn_id.clone(), r);
-                            }
+
+                    if s.is_done() && !s.mark_device_as_verified().await? {
+                        if let Some(r) = s.cancel() {
+                            self.outgoing_to_device_messages.insert(r.txn_id.clone(), r);
                         }
                     }
                 };
diff --git a/matrix_sdk_crypto/src/verification/sas/mod.rs b/matrix_sdk_crypto/src/verification/sas/mod.rs
index 38462f6e..6ab639b6 100644
--- a/matrix_sdk_crypto/src/verification/sas/mod.rs
+++ b/matrix_sdk_crypto/src/verification/sas/mod.rs
@@ -162,10 +162,8 @@ impl Sas {
             (content, guard.is_done())
         };
 
-        if done {
-            if !self.mark_device_as_verified().await? {
-                return Ok(self.cancel());
-            }
+        if done && !self.mark_device_as_verified().await? {
+            return Ok(self.cancel());
         }
 
         Ok(content.map(|c| {