Use QStringList::join(QChar) overload where applicable [QtSql]
authorMarc Mutz <marc.mutz@kdab.com>
Fri, 18 May 2012 18:00:23 +0000 (20:00 +0200)
committerQt by Nokia <qt-info@nokia.com>
Mon, 3 Sep 2012 17:35:25 +0000 (19:35 +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: Ie395d82d17710683968d006d22de313ef49dc6e5
Reviewed-by: Harald Fernengel <harald.fernengel@nokia.com>
Reviewed-by: Mark Brand <mabrand@mabrand.nl>
src/sql/drivers/odbc/qsql_odbc.cpp
src/sql/kernel/qsqldatabase.cpp
tests/auto/sql/kernel/qsqldatabase/tst_qsqldatabase.cpp

index f874374..c902f1c 100644 (file)
@@ -2260,7 +2260,7 @@ QStringList QODBCDriver::tables(QSql::TableType type) const
     if (tableType.isEmpty())
         return tl;
 
-    QString joinedTableTypeString = tableType.join(QLatin1String(","));
+    QString joinedTableTypeString = tableType.join(QLatin1Char(','));
 
     r = SQLTables(hStmt,
                    NULL,
index f0fa5c7..e382eef 100644 (file)
@@ -784,7 +784,7 @@ void QSqlDatabasePrivate::init(const QString &type)
     if (!driver) {
         qWarning("QSqlDatabase: %s driver not loaded", type.toLatin1().data());
         qWarning("QSqlDatabase: available drivers: %s",
-                        QSqlDatabase::drivers().join(QLatin1String(" ")).toLatin1().data());
+                        QSqlDatabase::drivers().join(QLatin1Char(' ')).toLatin1().data());
         if (QCoreApplication::instance() == 0)
             qWarning("QSqlDatabase: an instance of QCoreApplication is required for loading driver plugins");
         driver = shared_null()->driver;
index fb2db21..ce7b4e6 100644 (file)
@@ -431,7 +431,7 @@ void tst_QSqlDatabase::generic_data(const QString& engine)
 void tst_QSqlDatabase::addDatabase()
 {
     QTest::ignoreMessage(QtWarningMsg, "QSqlDatabase: BLAH_FOO_NONEXISTENT_DRIVER driver not loaded");
-    QTest::ignoreMessage(QtWarningMsg, qPrintable("QSqlDatabase: available drivers: " + QSqlDatabase::drivers().join(QLatin1String(" "))));
+    QTest::ignoreMessage(QtWarningMsg, qPrintable("QSqlDatabase: available drivers: " + QSqlDatabase::drivers().join(QLatin1Char(' '))));
     {
         QSqlDatabase db = QSqlDatabase::addDatabase("BLAH_FOO_NONEXISTENT_DRIVER",
                                                     "INVALID_CONNECTION");