Constify config file QFileInfo, create directly in if statement
authorClaudio Cambra <claudio.cambra@nextcloud.com>
Thu, 15 Dec 2022 12:24:34 +0000 (13:24 +0100)
committerClaudio Cambra <claudio.cambra@gmail.com>
Thu, 15 Dec 2022 15:58:00 +0000 (16:58 +0100)
Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
src/gui/accountmanager.cpp

index 2a96a3a60ad81a3a5fb3f8e7e7eb4203a807985c..95ba39b6305cadbc7080232209e5e6935594b961 100644 (file)
@@ -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();