From 87003c5923980554c06c3606009cd3022a771f0c Mon Sep 17 00:00:00 2001
From: abraunegg <alex.braunegg@gmail.com>
Date: Fri, 25 Nov 2022 06:57:30 +1100
Subject: [PATCH] Update when application exit message is printed (#2239)

* Update #2229 so that the exit message only appears when --synchronize is actually used
---
 src/main.d | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/main.d b/src/main.d
index 96651128..32ba4c15 100644
--- a/src/main.d
+++ b/src/main.d
@@ -56,7 +56,7 @@ int main(string[] args)
 	bool displayMemoryUsage = false;
 	bool displaySyncOptions = false;
 	bool cleanupLocalFilesGlobal = false;
-	bool monitorConfigured = false;
+	bool synchronizeConfigured = false;
 	
 	// start and finish messages
 	string startMessage = "Starting a sync with OneDrive";
@@ -70,7 +70,7 @@ int main(string[] args)
 	scope(exit) {
 		// detail what scope was called
 		log.vdebug("Exit scope called");
-		if (!monitorConfigured) {
+		if (synchronizeConfigured) {
 			log.log(finishMessage);
 		}
 		// Display memory details
@@ -835,9 +835,6 @@ int main(string[] args)
 		} else {
 			// Running as --monitor
 			log.error("Unable to reach Microsoft OneDrive API service at this point in time, re-trying network tests\n");
-			// set flag for exit scope
-			monitorConfigured = true;
-
 			// re-try network connection to OneDrive
 			// https://github.com/abraunegg/onedrive/issues/1184
 			// Back off & retry with incremental delay
@@ -1340,6 +1337,9 @@ int main(string[] args)
 
 		if (cfg.getValueBool("synchronize")) {
 			if (online) {
+				// set flag for exit scope
+				synchronizeConfigured = true;
+				
 				// Check user entry for local path - the above chdir means we are already in ~/OneDrive/ thus singleDirectory is local to this path
 				if (cfg.getValueString("single_directory") != "") {
 					// Does the directory we want to sync actually exist?