From: Claudio Cambra <claudio.cambra@nextcloud.com>
Date: Thu, 15 Dec 2022 12:24:34 +0000 (+0100)
Subject: Constify config file QFileInfo, create directly in if statement
X-Git-Tag: archive/raspbian/3.16.7-1_deb13u1+rpi1~1^2~12^2~11^2~59^2~4
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=d8e8a43cff26f7bfc03850cd04dd41fd26f3907e;p=nextcloud-desktop.git

Constify config file QFileInfo, create directly in if statement

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
---

diff --git a/src/gui/accountmanager.cpp b/src/gui/accountmanager.cpp
index 2a96a3a60..95ba39b63 100644
--- a/src/gui/accountmanager.cpp
+++ b/src/gui/accountmanager.cpp
@@ -172,11 +172,12 @@ bool AccountManager::restoreFromLegacySettings()
                                                       legacyCfgFileGrandParentFolder + legacyCfgFileRelativePath};
 
         for (const auto &configFile : legacyLocations) {
-            if (QFileInfo::exists(configFile)) {
+            if (const QFileInfo configFileInfo(configFile);
+                    configFileInfo.exists()) {
+
                 qCInfo(lcAccountManager) << "Migrate: checking old config " << configFile;
 
-                QFileInfo fi(configFile);
-                if (fi.isReadable()) {
+                if (configFileInfo.isReadable()) {
                     std::unique_ptr<QSettings> oCSettings(new QSettings(configFile, QSettings::IniFormat));
                     if (oCSettings->status() != QSettings::Status::NoError) {
                         qCInfo(lcAccountManager) << "Error reading legacy configuration file" << oCSettings->status();