diff options
author | Marc Mutz <marc.mutz@kdab.com> | 2017-04-07 12:44:01 +0200 |
---|---|---|
committer | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2017-04-07 14:35:04 +0000 |
commit | 25d1bbb84939b6960f16e82254fc844839ddfa54 (patch) | |
tree | 619233c312d38b30dd4242db957b7a9c612d2fd1 /examples/embedded | |
parent | f497dea73080c2d8910bd5b6d74c9a6226db0c92 (diff) |
examples: remove some unneeded QStringRef::toString() calls
- when appending to another string
- when just comparing
- when just calling toInt()
Change-Id: I6960784569b83bfd3e3640a8c04f2f909c293449
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Diffstat (limited to 'examples/embedded')
-rw-r--r-- | examples/embedded/flightinfo/flightinfo.cpp | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/examples/embedded/flightinfo/flightinfo.cpp b/examples/embedded/flightinfo/flightinfo.cpp index e91277db58..f46fc9693f 100644 --- a/examples/embedded/flightinfo/flightinfo.cpp +++ b/examples/embedded/flightinfo/flightinfo.cpp @@ -270,19 +270,18 @@ private: inFlightStatus |= className == "FlightDetailHeaderStatus"; inFlightMap |= className == "flightMap"; if (xml.name() == "td" && !className.isEmpty()) { - QString cn = className.toString(); - if (cn.contains("fieldTitle")) { + if (className.contains("fieldTitle")) { inFieldName = true; fieldNames += QString(); fieldValues += QString(); } - if (cn.contains("fieldValue")) + if (className.contains("fieldValue")) inFieldValue = true; } if (xml.name() == "img" && inFlightMap) { - QString src = xml.attributes().value("src").toString(); - src.prepend("http://mobile.flightview.com/"); - QUrl url = QUrl::fromPercentEncoding(src.toLatin1()); + const QByteArray encoded + = ("http://mobile.flightview.com/" % xml.attributes().value("src")).toLatin1(); + QUrl url = QUrl::fromPercentEncoding(encoded); mapReplies.append(m_manager.get(QNetworkRequest(url))); } } |