diff options
| author | Anton Kudryavtsev <a.kudryavtsev@netris.ru> | 2016-10-17 18:54:26 +0300 |
|---|---|---|
| committer | Anton Kudryavtsev <a.kudryavtsev@netris.ru> | 2016-10-18 17:52:04 +0000 |
| commit | 6a7b683817e7427cf952130b7726833fd5be74ce (patch) | |
| tree | 10a707cb680672e6d73af3e808a67d0d454871bb /src/plugins/printsupport/cups | |
| parent | a769b347542060e7d7e010c343d4a7d5c1549de0 (diff) | |
Plugins: optimize string usage
Prefer QStringRef methods to avoid allocations.
Use startsWith/endsWith rather than comparing substrings;
and avoid substrings where references suffice.
Use new QStringList::join(QL1S).
Change-Id: I46c44aca96578633370006d613eb0ac13f7cfc03
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Diffstat (limited to 'src/plugins/printsupport/cups')
| -rw-r--r-- | src/plugins/printsupport/cups/qppdprintdevice.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/printsupport/cups/qppdprintdevice.cpp b/src/plugins/printsupport/cups/qppdprintdevice.cpp index caa7382462..9efa83d409 100644 --- a/src/plugins/printsupport/cups/qppdprintdevice.cpp +++ b/src/plugins/printsupport/cups/qppdprintdevice.cpp @@ -63,7 +63,7 @@ QPpdPrintDevice::QPpdPrintDevice(const QString &id) if (!id.isEmpty()) { // TODO For now each dest is an individual device - QStringList parts = id.split(QLatin1Char('/')); + const auto parts = id.splitRef(QLatin1Char('/')); m_cupsName = parts.at(0).toUtf8(); if (parts.size() > 1) m_cupsInstance = parts.at(1).toUtf8(); |