AuroraMiddleware/qt5base-lts
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Avoid expensive QHash::values() calls

Browse Source 
qcocoawindow.mm: we can replace QHash::values() with std::vector
since CoW is needless here and std::vector is more cache-friendly.
Also replace foreach with range-based for.

qitemeditorfactory.cpp: QHash::values() is used as auxiliary container
to create QSet. Replace it with std::vector since CoW is needless here
and apply sort-unique idiom to remove duplicates.
Also avoid needless allocations.

Change-Id: If34c7016977ceb7fab68e9298bf2e1944af79139
Reviewed-by: Marc Mutz <marc.mutz@kdab.com>
This commit is contained in:
Anton Kudryavtsev 2017-03-19 14:19:22 +03:00
parent 81b5aa792f
commit bae0c4c11a
2 changed files with 10 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/plugins/platforms/cocoa/qcocoawindow.mm
View File

@ -58,6 +58,8 @@
#include <QDebug>
#include <vector>
enum {
defaultWindowWidth = 160,
defaultWindowHeight = 160
@ -2068,10 +2070,10 @@ void QCocoaWindow::applyContentBorderThickness(NSWindow *window)
}
// Find consecutive registered border areas, starting from the top.
QList<BorderRange> ranges = m_contentBorderAreas.values();
std::vector<BorderRange> ranges(m_contentBorderAreas.cbegin(), m_contentBorderAreas.cend());
std::sort(ranges.begin(), ranges.end());
int effectiveTopContentBorderThickness = m_topContentBorderThickness;
foreach (BorderRange range, ranges) {
for (BorderRange range : ranges) {
// Skip disiabled ranges (typically hidden tool bars)
if (!m_enabledContentBorderAreas.value(range.identifier, false))
continue;

9
src/widgets/itemviews/qitemeditorfactory.cpp
View File

@ -55,6 +55,7 @@
#include <qapplication.h>
#include <qdebug.h>
#include <vector>
#include <algorithm>
QT_BEGIN_NAMESPACE
@ -191,9 +192,11 @@ QByteArray QItemEditorFactory::valuePropertyName(int userType) const
QItemEditorFactory::~QItemEditorFactory()
{
//we make sure we delete all the QItemEditorCreatorBase
//this has to be done only once, hence the QSet
QSet<QItemEditorCreatorBase*> set = creatorMap.values().toSet();
qDeleteAll(set);
//this has to be done only once, hence the sort-unique idiom
std::vector<QItemEditorCreatorBase*> creators(creatorMap.cbegin(), creatorMap.cend());
std::sort(creators.begin(), creators.end());
const auto it = std::unique(creators.begin(), creators.end());
qDeleteAll(creators.begin(), it);
}
/*!