From a8a5e184abc94146cfa6f6c7a03486c43c52e14d Mon Sep 17 00:00:00 2001
From: Hocuri <hocuri@gmx.de>
Date: Fri, 15 Apr 2022 21:30:01 +0200
Subject: [PATCH] Don't unnecessarily interrupt ephemeral loop (#3221)

Follow-up for #3211
---
 src/imap.rs | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/imap.rs b/src/imap.rs
index 6e2f27a3b..16ef260ee 100644
--- a/src/imap.rs
+++ b/src/imap.rs
@@ -458,13 +458,15 @@ impl Imap {
         }
         self.prepare(context).await?;
 
-        if self
+        let msgs_fetched = self
             .fetch_new_messages(context, watch_folder, false)
             .await
-            .context("fetch_new_messages")?
-        {
-            // New messages were fetched, restart ephemeral loop in case these messages will expire
-            // later.
+            .context("fetch_new_messages")?;
+        if msgs_fetched && context.get_config_delete_device_after().await?.is_some() {
+            // New messages were fetched and shall be deleted later, restart ephemeral loop.
+            // Note that the `Config::DeleteDeviceAfter` timer starts as soon as the messages are
+            // fetched while the per-chat ephemeral timers start as soon as the messages are marked
+            // as noticed.
             context.interrupt_ephemeral_task().await;
         }