Use QStringList::join(QChar) overload where applicable [QtNetwork]
authorMarc Mutz <marc.mutz@kdab.com>
Fri, 18 May 2012 18:00:23 +0000 (20:00 +0200)
committerQt by Nokia <qt-info@nokia.com>
Fri, 7 Sep 2012 16:41:22 +0000 (18:41 +0200)
This is an automated change performing the following replacements:
                    join\("(.)"\) -> join('\1')
  join\(QLatin1String\("(.)"\)\)  -> join(QLatin1Char('\1'))
  join\(QStringLiteral\("(.)"\)\) -> join(QLatin1Char('\1'))

Change-Id: I5b75877ba192fa1357e67fee70dff7c0475991e8
Reviewed-by: João Abecasis <joao.abecasis@nokia.com>
Reviewed-by: Shane Kearns <shane.kearns@accenture.com>
src/network/kernel/qnetworkproxy.cpp
tests/auto/network/kernel/qdnslookup/tst_qdnslookup.cpp
tests/auto/network/kernel/qhostinfo/tst_qhostinfo.cpp

index e26374e..181ea51 100644 (file)
@@ -1610,7 +1610,7 @@ QDebug operator<<(QDebug debug, const QNetworkProxy &proxy)
         scaps << QStringLiteral("Caching");
     if (caps & QNetworkProxy::HostNameLookupCapability)
         scaps << QStringLiteral("NameLookup");
-    debug << "[" << scaps.join(QStringLiteral(" ")) << "]";
+    debug << "[" << scaps.join(QLatin1Char(' ')) << "]";
     return debug;
 }
 #endif
index 0afdd56..c80f2a8 100644 (file)
@@ -168,7 +168,7 @@ void tst_QDnsLookup::lookup()
             addresses << record.value().toString().toLower();
     }
     addresses.sort();
-    QCOMPARE(addresses.join(" "), host);
+    QCOMPARE(addresses.join(' '), host);
 
     // mail exchanges
     QStringList mailExchanges;
@@ -176,7 +176,7 @@ void tst_QDnsLookup::lookup()
         QCOMPARE(record.name(), domain);
         mailExchanges << QString("%1 %2").arg(QString::number(record.preference()), record.exchange());
     }
-    QCOMPARE(mailExchanges.join(" "), mx);
+    QCOMPARE(mailExchanges.join(' '), mx);
 
     // name servers
     QStringList nameServers;
@@ -186,7 +186,7 @@ void tst_QDnsLookup::lookup()
             nameServers << record.value();
     }
     nameServers.sort();
-    QCOMPARE(nameServers.join(" "), ns);
+    QCOMPARE(nameServers.join(' '), ns);
 
     // pointers
     if (!ptr.isEmpty()) {
@@ -208,7 +208,7 @@ void tst_QDnsLookup::lookup()
                 QString::number(record.port()),
                 record.target());
     }
-    QCOMPARE(services.join(" "), srv);
+    QCOMPARE(services.join(' '), srv);
 
     // text
     if (!txt.isEmpty()) {
index 5285425..a733385 100644 (file)
@@ -302,7 +302,7 @@ void tst_QHostInfo::lookupIPv4()
     QStringList expected = addresses.split(' ');
     expected.sort();
 
-    QCOMPARE(tmp.join(" "), expected.join(" "));
+    QCOMPARE(tmp.join(' '), expected.join(' '));
 }
 
 void tst_QHostInfo::lookupIPv6_data()
@@ -349,7 +349,7 @@ void tst_QHostInfo::lookupIPv6()
     QStringList expected = addresses.split(' ');
     expected.sort();
 
-    QCOMPARE(tmp.join(" ").toLower(), expected.join(" ").toLower());
+    QCOMPARE(tmp.join(' ').toLower(), expected.join(' ').toLower());
 }
 
 void tst_QHostInfo::reverseLookup_data()
@@ -414,7 +414,7 @@ void tst_QHostInfo::blockingLookup()
     QStringList expected = addresses.split(' ');
     expected.sort();
 
-    QCOMPARE(tmp.join(" ").toUpper(), expected.join(" ").toUpper());
+    QCOMPARE(tmp.join(' ').toUpper(), expected.join(' ').toUpper());
 }
 
 void tst_QHostInfo::raceCondition()