Fix review comments.

Signed-off-by: alex-z <[email protected]>
nextcloud · Jul 17, 2023 · 50dd564 · 50dd564
1 parent 219275e
commit 50dd564
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 8 deletions.
diff --git a/src/gui/tray/activitylistmodel.cpp b/src/gui/tray/activitylistmodel.cpp
@@ -440,7 +440,6 @@ void ActivityListModel::startFetchJob()
 
 void ActivityListModel::setFinalList(const ActivityList &finalList)
 {
-    QMutexLocker locker(&_finalListMutex);
     _finalList = finalList;
 
     emit allConflictsChanged();
@@ -670,7 +669,6 @@ void ActivityListModel::removeActivityFromActivityList(const Activity &activity)
     qCInfo(lcActivity) << "Trying to remove Activity/Notification/Error from view... ";
 
     {
-        QMutexLocker locker(&_finalListMutex);
         const auto index = _finalList.indexOf(activity);
         if (index != -1) {
             qCInfo(lcActivity) << "Activity/Notification/Error successfully removed from the list.";
@@ -697,7 +695,6 @@ void ActivityListModel::checkAndRemoveSeenActivities(const OCC::ActivityList &ne
 {
     ActivityList activitiesToRemove;
     {
-        QMutexLocker locker(&_finalListMutex);
         for (const auto &activity : _finalList) {
             if (activity._objectType == QStringLiteral("chat") && !newActivities.contains(activity)) {
                 activitiesToRemove.push_back(activity);
@@ -961,7 +958,6 @@ void ActivityListModel::slotRefreshActivityInitial()
 void ActivityListModel::slotRemoveAccount()
 {
     {
-        QMutexLocker locker(&_finalListMutex);
         _finalList.clear();
     }
     _activityLists.clear();

diff --git a/src/gui/tray/activitylistmodel.h b/src/gui/tray/activitylistmodel.h
@@ -191,7 +191,6 @@ private slots:
     ActivityList _listOfIgnoredFiles;
     ActivityList _notificationErrorsLists;
     ActivityList _finalList;
-    QMutex _finalListMutex;
 
     QSet<qint64> _presentedActivities;
 

diff --git a/src/gui/tray/usermodel.cpp b/src/gui/tray/usermodel.cpp
@@ -245,7 +245,6 @@ void User::slotBuildNotificationDisplay(const ActivityList &list)
 
 void User::slotNotificationFetchFinished()
 {
-    QMutexLocker locker(&_isNotificationFetchRunningMutex);
     _isNotificationFetchRunning = false;
 }
 
@@ -451,7 +450,6 @@ void User::slotRefreshNotifications()
     // start a server notification handler if no notification requests
     // are running
     if (_notificationRequestsRunning == 0) {
-        QMutexLocker locker(&_isNotificationFetchRunningMutex);
         if (_isNotificationFetchRunning) {
             qCDebug(lcActivity) << "Notification fetch is already running.";
             return;

diff --git a/src/gui/tray/usermodel.h b/src/gui/tray/usermodel.h
@@ -188,7 +188,6 @@ private slots:
 
     int _lastChatNotificationsReceivedCount = 0;
 
-    QMutex _isNotificationFetchRunningMutex;
     bool _isNotificationFetchRunning = false;
 };