[geary/wip/778276-better-flag-updates] Turn some debug logging down a bit.



commit 5446cc25f4bb6a91078a2bb646d6f18d07104354
Author: Michael James Gratton <mike vee net>
Date:   Wed Dec 20 21:02:18 2017 +1030

    Turn some debug logging down a bit.

 src/engine/app/app-email-store.vala  |   14 +-------------
 src/engine/imap/api/imap-folder.vala |    7 +++----
 2 files changed, 4 insertions(+), 17 deletions(-)
---
diff --git a/src/engine/app/app-email-store.vala b/src/engine/app/app-email-store.vala
index 4390dfa..e616755 100644
--- a/src/engine/app/app-email-store.vala
+++ b/src/engine/app/app-email-store.vala
@@ -179,9 +179,6 @@ public class Geary.App.EmailStore : BaseObject {
             bool open = false;
             Gee.Collection<Geary.EmailIdentifier>? used_ids = null;
             try {
-                debug("EmailStore opening %s for %s on %d emails", folder.to_string(),
-                    operation.get_type().name(), ids.size);
-
                 yield folder.open_async(Geary.Folder.OpenFlags.FAST_OPEN, cancellable);
                 open = true;
                 used_ids = yield operation.execute_async(folder, ids, cancellable);
@@ -193,12 +190,6 @@ public class Geary.App.EmailStore : BaseObject {
                         // Don't use the cancellable here, if it's been
                         // opened we need to try to close it.
                         yield folder.close_async(null);
-                        debug(
-                            "EmailStore closed %s after %s on %d emails",
-                            folder.to_string(),
-                            operation.get_type().name(),
-                            ids.size
-                        );
                     } catch (Error e) {
                         debug("Error closing folder %s: %s",
                               folder.to_string(), e.message);
@@ -216,10 +207,7 @@ public class Geary.App.EmailStore : BaseObject {
             // And we don't want to operate on the same folder twice.
             folders_to_ids.remove_all(path);
         }
-        
-        debug("EmailStore %s done running %s on %d emails", account.to_string(),
-            operation.get_type().name(), emails.size);
-        
+
         if (folders_to_ids.size > 0) {
             debug("Couldn't perform %s on some messages in %s", operation.get_type().name(),
                 account.to_string());
diff --git a/src/engine/imap/api/imap-folder.vala b/src/engine/imap/api/imap-folder.vala
index 69a2bd8..a58ea2a 100644
--- a/src/engine/imap/api/imap-folder.vala
+++ b/src/engine/imap/api/imap-folder.vala
@@ -212,16 +212,15 @@ private class Geary.Imap.Folder : BaseObject {
 
     private void on_fetch(FetchedData data) {
         // add if not found, merge if already received data for this email
-        debug("%s: FETCH (%s): %s:",
-              to_string(),
-              this.fetch_accumulator != null ? "accumulating" : "unsolicited",
-              data.to_string());
         if (this.fetch_accumulator != null) {
             FetchedData? existing = this.fetch_accumulator.get(data.seq_num);
             this.fetch_accumulator.set(
                 data.seq_num, (existing != null) ? data.combine(existing) : data
             );
         } else {
+            debug("%s: FETCH (unsolicited): %s:",
+                  to_string(),
+                  data.to_string());
             updated(data.seq_num, data);
         }
     }


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