diff options
| author | Marc Mutz <marc.mutz@kdab.com> | 2014-03-27 17:30:10 +0100 |
|---|---|---|
| committer | The Qt Project <gerrit-noreply@qt-project.org> | 2014-03-29 12:31:42 +0100 |
| commit | 396e3a18d766c51f4422c5729a4baba1fb1538fa (patch) | |
| tree | b2051952f7e646fd14eedeadde0a00930e83d0a1 /src/plugins/platforms/qnx | |
| parent | e009a3156b8022c9d2909d24f8aaefc04c66e24e (diff) | |
QNX: don't copy a QStringList twice every loop iteration
Apparently, the function before returned const-&, but now returns by-value.
Use the member data variable directly, the function is const anyway.
Change-Id: I324bc675fd7c9b5bfb28c475eba0c0275379132c
Reviewed-by: Rafael Roquetto <rafael.roquetto@kdab.com>
Diffstat (limited to 'src/plugins/platforms/qnx')
| -rw-r--r-- | src/plugins/platforms/qnx/qqnxfilepicker.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/plugins/platforms/qnx/qqnxfilepicker.cpp b/src/plugins/platforms/qnx/qqnxfilepicker.cpp index 830b110f2a..f38b2dee88 100644 --- a/src/plugins/platforms/qnx/qqnxfilepicker.cpp +++ b/src/plugins/platforms/qnx/qqnxfilepicker.cpp @@ -288,8 +288,8 @@ QString QQnxFilePicker::filePickerType() const bool video = false; bool music = false; QMimeDatabase mimeDb; - for (int i = 0; i < filters().count(); i++) { - QList<QMimeType> mimeTypes = mimeDb.mimeTypesForFileName(filters().at(i)); + for (int i = 0; i < m_filters.count(); i++) { + QList<QMimeType> mimeTypes = mimeDb.mimeTypesForFileName(m_filters.at(i)); if (mimeTypes.isEmpty()) return QStringLiteral("Other"); |