[geary/mjog/741-inbox-getting-closed: 87/91] Geary.App.ConversationMonitor: Throw error closing folder when possible



commit a385ce8fc2cc54371af37749986d94dcb9bb1bc7
Author: Michael Gratton <mike vee net>
Date:   Fri Apr 10 14:41:00 2020 +1000

    Geary.App.ConversationMonitor: Throw error closing folder when possible

 src/engine/app/app-conversation-monitor.vala | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)
---
diff --git a/src/engine/app/app-conversation-monitor.vala b/src/engine/app/app-conversation-monitor.vala
index 644605d4..40cddfa5 100644
--- a/src/engine/app/app-conversation-monitor.vala
+++ b/src/engine/app/app-conversation-monitor.vala
@@ -681,10 +681,16 @@ public class Geary.App.ConversationMonitor : BaseObject, Logging.Source {
         bool closing = false;
         if (folder_was_opened) {
             try {
-                // Always close the folder to prevent open leaks
                 closing = yield this.base_folder.close_async(null);
-            } catch (Error err) {
-                warning("Unable to close monitored folder: %s", err.message);
+            } catch (GLib.Error err) {
+                if (close_err == null) {
+                    close_err = err;
+                } else {
+                    warning(
+                        "Unable to close monitored folder %s: %s",
+                        this.base_folder.to_string(), err.message
+                    );
+                }
             }
         }
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]