From: Harald Eilertsen <haraldei@anduin.net>
Date: Tue, 29 Sep 2020 16:32:48 +0000 (+0200)
Subject: Don't hold ref to value returned from function.
X-Git-Tag: archive/raspbian/3.16.7-1_deb13u1+rpi1~1^2~12^2~22^2~137^2~1
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=061ef027ff89641bcdecf4bc559c38089b33a62b;p=nextcloud-desktop.git

Don't hold ref to value returned from function.

Signed-off-by: Harald Eilertsen <haraldei@anduin.net>
---

diff --git a/src/gui/accountsettings.cpp b/src/gui/accountsettings.cpp
index 62491c5db..49e90e19b 100644
--- a/src/gui/accountsettings.cpp
+++ b/src/gui/accountsettings.cpp
@@ -761,7 +761,7 @@ void AccountSettings::slotAccountStateChanged()
         AccountPtr account = _accountState->account();
         QUrl safeUrl(account->url());
         safeUrl.setPassword(QString()); // Remove the password from the URL to avoid showing it in the UI
-        const auto &folders = FolderMan::instance()->map().values();
+        const auto folders = FolderMan::instance()->map().values();
         for (Folder *folder : folders) {
             _model->slotUpdateFolderState(folder);
         }
@@ -894,14 +894,14 @@ void AccountSettings::refreshSelectiveSyncStatus()
 
     QString msg;
     int cnt = 0;
-    const auto &folders = FolderMan::instance()->map().values();
+    const auto folders = FolderMan::instance()->map().values();
     for (Folder *folder : folders) {
         if (folder->accountState() != _accountState) {
             continue;
         }
 
         bool ok = false;
-        const auto &undecidedList = folder->journalDb()->getSelectiveSyncList(SyncJournalDb::SelectiveSyncUndecidedList, &ok);
+        const auto undecidedList = folder->journalDb()->getSelectiveSyncList(SyncJournalDb::SelectiveSyncUndecidedList, &ok);
         QString p;
         for (const auto &it : undecidedList) {
             // FIXME: add the folder alias in a hoover hint.
diff --git a/src/gui/accountstate.cpp b/src/gui/accountstate.cpp
index be8b90876..58a679031 100644
--- a/src/gui/accountstate.cpp
+++ b/src/gui/accountstate.cpp
@@ -441,7 +441,7 @@ void AccountState::slotNavigationAppsFetched(const QJsonDocument &reply, int sta
 
             if(!reply.isEmpty()){
                 auto element = reply.object().value("ocs").toObject().value("data");
-                const auto &navLinks = element.toArray();
+                const auto navLinks = element.toArray();
 
                 if(navLinks.size() > 0){
                     for (const QJsonValue &value : navLinks) {
@@ -468,7 +468,7 @@ AccountAppList AccountState::appList() const
 AccountApp* AccountState::findApp(const QString &appId) const
 {
     if(!appId.isEmpty()) {
-        const auto &apps = appList();
+        const auto apps = appList();
         const auto it = std::find_if(apps.cbegin(), apps.cend(), [appId](const auto &app) {
             return app->id() == appId;
         });
diff --git a/src/gui/application.cpp b/src/gui/application.cpp
index ca42c0cc4..9ac569cbc 100644
--- a/src/gui/application.cpp
+++ b/src/gui/application.cpp
@@ -349,7 +349,7 @@ void Application::slotSystemOnlineConfigurationChanged(QNetworkConfiguration cnf
 
 void Application::slotCheckConnection()
 {
-    const auto &list = AccountManager::instance()->accounts();
+    const auto list = AccountManager::instance()->accounts();
     for (const auto &accountState : list) {
         AccountState::State state = accountState->state();
 
diff --git a/src/gui/folder.cpp b/src/gui/folder.cpp
index 85bfcee4d..b4bfab3e6 100644
--- a/src/gui/folder.cpp
+++ b/src/gui/folder.cpp
@@ -528,7 +528,7 @@ void Folder::saveToSettings() const
     // This ensures that older clients will not read a configuration
     // where two folders for different accounts point at the same
     // local folders.
-    const auto &folderMap = FolderMan::instance()->map();
+    const auto folderMap = FolderMan::instance()->map();
     const auto it = std::find_if(folderMap.cbegin(), folderMap.cend(), [this](const auto *other) {
         return other != this && other->cleanPath() == this->cleanPath();
     });
diff --git a/src/gui/folderman.cpp b/src/gui/folderman.cpp
index 472be0663..92a842488 100755
--- a/src/gui/folderman.cpp
+++ b/src/gui/folderman.cpp
@@ -280,7 +280,7 @@ int FolderMan::setupFoldersMigration()
     QDir dir(_folderConfigPath);
     //We need to include hidden files just in case the alias starts with '.'
     dir.setFilter(QDir::Files | QDir::Hidden);
-    const auto &list = dir.entryList();
+    const auto list = dir.entryList();
 
     // Normally there should be only one account when migrating.
     AccountState *accountState = AccountManager::instance()->accounts().value(0).data();
@@ -913,7 +913,7 @@ Folder *FolderMan::addFolder(AccountState *accountState, const FolderDefinition
 
     // Migration: The first account that's configured for a local folder shall
     // be saved in a backwards-compatible way.
-    const auto &folderList = FolderMan::instance()->map();
+    const auto folderList = FolderMan::instance()->map();
     const auto it = std::find_if(folderList.cbegin(), folderList.cend(), [this, folder](const auto *other) {
         return other != folder && other->cleanPath() == folder->cleanPath();
     });
@@ -975,7 +975,7 @@ Folder *FolderMan::folderForPath(const QString &path)
 {
     QString absolutePath = QDir::cleanPath(path) + QLatin1Char('/');
 
-    const auto &folders = this->map().values();
+    const auto folders = this->map().values();
     const auto it = std::find_if(folders.cbegin(), folders.cend(), [absolutePath](const auto *folder) {
         const QString folderPath = folder->cleanPath() + QLatin1Char('/');
         return absolutePath.startsWith(folderPath, (Utility::isWindows() || Utility::isMac()) ? Qt::CaseInsensitive : Qt::CaseSensitive);