Fix Coverity Scan 143909.

Also, the setting "Confirmation on auto-exit when downloads finish" wasn't working before.

src/app/application.cpp

@@ -285,38 +285,32 @@ void Application::allTorrentsFinished()
 {
 #ifndef DISABLE_GUI
     Preferences *const pref = Preferences::instance();
+    bool isExit = pref->shutdownqBTWhenDownloadsComplete();
+    bool isShutdown = pref->shutdownWhenDownloadsComplete();
+    bool isSuspend = pref->suspendWhenDownloadsComplete();
+    bool isHibernate = pref->hibernateWhenDownloadsComplete();
 
-    bool will_shutdown = (pref->shutdownWhenDownloadsComplete()
-                          || pref->shutdownqBTWhenDownloadsComplete()
-                          || pref->suspendWhenDownloadsComplete()
-                          || pref->hibernateWhenDownloadsComplete());
+    bool haveAction = isExit || isShutdown || isSuspend || isHibernate;
+    if (!haveAction) return;
 
-    // Auto-Shutdown
-    if (will_shutdown) {
-        bool suspend = pref->suspendWhenDownloadsComplete();
-        bool hibernate = pref->hibernateWhenDownloadsComplete();
-        bool shutdown = pref->shutdownWhenDownloadsComplete();
-
-        // Confirm shutdown
     ShutdownDialogAction action = ShutdownDialogAction::Exit;
-        if (suspend)
+    if (isSuspend)
         action = ShutdownDialogAction::Suspend;
-        else if (hibernate)
+    else if (isHibernate)
         action = ShutdownDialogAction::Hibernate;
-        else if (shutdown)
+    else if (isShutdown)
         action = ShutdownDialogAction::Shutdown;
 
-        if ((action == ShutdownDialogAction::Exit) && (!pref->dontConfirmAutoExit())) {
-            if (!ShutdownConfirmDlg::askForConfirmation(action))
-                return;
+    // ask confirm
+    if ((action == ShutdownDialogAction::Exit) && (pref->dontConfirmAutoExit())) {
+        // do nothing & skip confirm
     }
-        else { //exit and shutdown
-            if (!ShutdownConfirmDlg::askForConfirmation(action))
-                return;
+    else {
+        if (!ShutdownConfirmDlg::askForConfirmation(action)) return;
     }
 
     // Actually shut down
-        if (suspend || hibernate || shutdown) {
+    if (action != ShutdownDialogAction::Exit) {
         qDebug("Preparing for auto-shutdown because all downloads are complete!");
         // Disabling it for next time
         pref->setShutdownWhenDownloadsComplete(false);
@@ -325,9 +319,9 @@ void Application::allTorrentsFinished()
         // Make sure preferences are synced before exiting
         m_shutdownAct = action;
     }
+
     qDebug("Exiting the application");
     exit();
-    }
 #endif // DISABLE_GUI
 }