diff options
author | Anton Kudryavtsev <a.kudryavtsev@netris.ru> | 2016-03-31 12:01:59 +0300 |
---|---|---|
committer | Anton Kudryavtsev <a.kudryavtsev@netris.ru> | 2016-07-02 07:39:06 +0000 |
commit | 6662919ecde901771d9641fd732aa0735ebb39e6 (patch) | |
tree | 449752f7d9582cd685d81829e0e2245b0f9a0ece /src/corelib/plugin | |
parent | 2185b2f054ddae0fc67b8799b3561c57729f1cd7 (diff) |
CoreLib: use QStringRef to optimize memory allocation
Replace substring functions that return QString with
corresponding functions that return QStringRef where
it's possible.
Create QString from QStringRef only where necessary.
Change-Id: Id9ea11b16947220cd27787c0b529de62d10b6c26
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Diffstat (limited to 'src/corelib/plugin')
-rw-r--r-- | src/corelib/plugin/qlibrary.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/corelib/plugin/qlibrary.cpp b/src/corelib/plugin/qlibrary.cpp index f7e57461e3..886dd20d0e 100644 --- a/src/corelib/plugin/qlibrary.cpp +++ b/src/corelib/plugin/qlibrary.cpp @@ -616,12 +616,12 @@ bool QLibrary::isLibrary(const QString &fileName) QString completeSuffix = QFileInfo(fileName).completeSuffix(); if (completeSuffix.isEmpty()) return false; - QStringList suffixes = completeSuffix.split(QLatin1Char('.')); + const QVector<QStringRef> suffixes = completeSuffix.splitRef(QLatin1Char('.')); # if defined(Q_OS_DARWIN) // On Mac, libs look like libmylib.1.0.0.dylib - const QString lastSuffix = suffixes.at(suffixes.count() - 1); - const QString firstSuffix = suffixes.at(0); + const QStringRef &lastSuffix = suffixes.at(suffixes.count() - 1); + const QStringRef &firstSuffix = suffixes.at(0); bool valid = (lastSuffix == QLatin1String("dylib") || firstSuffix == QLatin1String("so") @@ -657,7 +657,7 @@ bool QLibrary::isLibrary(const QString &fileName) int suffix; int suffixPos = -1; for (suffix = 0; suffix < validSuffixList.count() && suffixPos == -1; ++suffix) - suffixPos = suffixes.indexOf(validSuffixList.at(suffix)); + suffixPos = suffixes.indexOf(QStringRef(&validSuffixList.at(suffix))); bool valid = suffixPos != -1; for (int i = suffixPos + 1; i < suffixes.count() && valid; ++i) |