From 5335a7153549fcb66f357c04b3b94e3ab112dc4e Mon Sep 17 00:00:00 2001 From: Marc Mutz Date: Fri, 22 Aug 2014 20:55:24 +0200 Subject: 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 --- src/gui/image/qiconloader.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'src/gui/image/qiconloader.cpp') 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(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)); } -- cgit v1.2.3