diff options
| author | Marc Mutz <marc.mutz@kdab.com> | 2014-08-22 20:55:24 +0200 |
|---|---|---|
| committer | Marc Mutz <marc.mutz@kdab.com> | 2014-08-23 13:18:15 +0200 |
| commit | 5335a7153549fcb66f357c04b3b94e3ab112dc4e (patch) | |
| tree | 0937b503c6f24f306ff3c8cb75d1e2b3eff01f29 /src/gui/image/qiconloader.cpp | |
| parent | 8f0f7acb67a009c487fb3778e20a7d82d95f6661 (diff) | |
QIconLoader: don't re-evaluate container.size() all the time
Experience shows that the compiler does not recognize the
size() call as a constant expression, so help it along.
Change-Id: I60cd1754f8be123c62e3b47f6f721f42a69fe3c5
Reviewed-by: Olivier Goffart <ogoffart@woboq.com>
Diffstat (limited to 'src/gui/image/qiconloader.cpp')
| -rw-r--r-- | src/gui/image/qiconloader.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/gui/image/qiconloader.cpp b/src/gui/image/qiconloader.cpp index 9c09c02c0c..88f505cc39 100644 --- a/src/gui/image/qiconloader.cpp +++ b/src/gui/image/qiconloader.cpp @@ -564,9 +564,11 @@ void QIconLoaderEngine::virtual_hook(int id, void *data) QIconEngine::AvailableSizesArgument &arg = *reinterpret_cast<QIconEngine::AvailableSizesArgument*>(data); arg.sizes.clear(); + const int N = m_entries.size(); + arg.sizes.reserve(N); // Gets all sizes from the DirectoryInfo entries - for (int i = 0 ; i < m_entries.size() ; ++i) { + for (int i = 0; i < N; ++i) { int size = m_entries.at(i)->dir.size; arg.sizes.append(QSize(size, size)); } |