From 7c0b5c08cc97eded7755ca5cef9c7e5ba5c15537 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Stefan=20Br=C3=BCns?= Date: Mon, 15 Feb 2021 17:34:01 +0100 Subject: [PATCH] [FileItemRolesUpdater] Reserve space in two temporary lists Reserve sufficient space to avoid reallocation later. Overallocating actually requires less memory than growing and copying it, and it is freed immediately after anyway. --- src/kitemviews/kfileitemmodelrolesupdater.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/kitemviews/kfileitemmodelrolesupdater.cpp b/src/kitemviews/kfileitemmodelrolesupdater.cpp index 0554bff60e..d18387f512 100644 --- a/src/kitemviews/kfileitemmodelrolesupdater.cpp +++ b/src/kitemviews/kfileitemmodelrolesupdater.cpp @@ -954,6 +954,8 @@ void KFileItemModelRolesUpdater::updateChangedItems() QList visibleChangedIndexes; QList invisibleChangedIndexes; + visibleChangedIndexes.reserve(m_changedItems.size()); + invisibleChangedIndexes.reserve(m_changedItems.size()); // Iterate over a const copy because items are deleted within the loop const auto changedItems = m_changedItems;