From c603681115b6071f241f6931685d36a92b6403f8 Mon Sep 17 00:00:00 2001
From: isanae <14251494+isanae@users.noreply.github.com>
Date: Thu, 19 Sep 2019 10:07:52 -0400
Subject: moved helper stuff to spawn so it can reuse error handling removed
 unused helper::init() removed logging when deleting a credential that doesn't
 exist, happens all the time

---
 src/settingsutilities.cpp | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

(limited to 'src/settingsutilities.cpp')

diff --git a/src/settingsutilities.cpp b/src/settingsutilities.cpp
index 6c99a602..db7c1818 100644
--- a/src/settingsutilities.cpp
+++ b/src/settingsutilities.cpp
@@ -224,16 +224,18 @@ bool deleteWindowsCredential(const QString& key)
 
   if (!CredDeleteW(credName.toStdWString().c_str(), CRED_TYPE_GENERIC, 0)) {
     const auto e = GetLastError();
+
+    // not an error if the key already doesn't exist, and don't log it because
+    // it happens all the time when the settings dialog is closed since it
+    // doesn't check first
     if (e == ERROR_NOT_FOUND) {
-      // not an error if the key already doesn't exist
-      log::debug("can't delete windows credential {}, doesn't exist", credName);
       return true;
-    } else {
-      log::error(
-        "failed to delete windows credential {}, {}",
-        credName, formatSystemMessage(e));
-      return false;
     }
+
+    log::error(
+      "failed to delete windows credential {}, {}",
+      credName, formatSystemMessage(e));
+    return false;
   }
 
   log::debug("deleted windows credential {}", credName);
@@ -269,7 +271,7 @@ bool addWindowsCredential(const QString& key, const QString& data)
     return false;
   }
 
-  log::debug("added windows credential {}", credName);
+  log::debug("set windows credential {}", credName);
 
   return true;
 }
-- 
cgit v1.3.1