clang-tidy: avoid copy, use const references

4 files changed, 4 insertions, 4 deletions

diff --git a/src/kitemviews/kfileitemmodelrolesupdater.cpp b/src/kitemviews/kfileitemmodelrolesupdater.cpp
index b2c4646a1..679370455 100644
--- a/src/kitemviews/kfileitemmodelrolesupdater.cpp
+++ b/src/kitemviews/kfileitemmodelrolesupdater.cpp
@@ -1526,7 +1526,7 @@ void KFileItemModelRolesUpdater::resetSizeData(const int index, const int size)
     connect(m_model, &KFileItemModel::itemsChanged, this, &KFileItemModelRolesUpdater::slotItemsChanged);
 }
 
-void KFileItemModelRolesUpdater::recountDirectoryItems(const QList<QUrl> directories)
+void KFileItemModelRolesUpdater::recountDirectoryItems(const QList<QUrl> &directories)
 {
     for (const auto &dir : directories) {
         auto index = m_model->index(dir);
diff --git a/src/kitemviews/kfileitemmodelrolesupdater.h b/src/kitemviews/kfileitemmodelrolesupdater.h
index 56e28ce72..aadcb66f8 100644
--- a/src/kitemviews/kfileitemmodelrolesupdater.h
+++ b/src/kitemviews/kfileitemmodelrolesupdater.h
@@ -342,7 +342,7 @@ private:
 
     void resetSizeData(const int index, const int size = 0);
 
-    void recountDirectoryItems(const QList<QUrl> directories);
+    void recountDirectoryItems(const QList<QUrl> &directories);
 
 private:
     QSize cacheSize();
diff --git a/src/kitemviews/kitemset.cpp b/src/kitemviews/kitemset.cpp
index c8c1ff076..c23ef55f5 100644
--- a/src/kitemviews/kitemset.cpp
+++ b/src/kitemviews/kitemset.cpp
@@ -94,7 +94,7 @@ KItemSet::iterator KItemSet::insert(int i)
     }
 }
 
-KItemSet::iterator KItemSet::erase(iterator it)
+KItemSet::iterator KItemSet::erase(const iterator &it)
 {
     KItemRangeList::iterator rangeIt = it.m_rangeIt;
 
diff --git a/src/kitemviews/kitemset.h b/src/kitemviews/kitemset.h
index 8dd134ec6..7e5fa4e6f 100644
--- a/src/kitemviews/kitemset.h
+++ b/src/kitemviews/kitemset.h
@@ -287,7 +287,7 @@ public:
     iterator find(int i);
     const_iterator constFind(int i) const;
     bool remove(int i);
-    iterator erase(iterator it);
+    iterator erase(const iterator &it);
 
     /**
      * Returns a new set which contains all items that are contained in this