From df9d882d41b741fef7c5beeddb0abe9d904443d8 Mon Sep 17 00:00:00 2001
From: Marc Mutz <marc.mutz@qt.io>
Date: Fri, 30 Sep 2022 14:09:04 +0200
Subject: Port from container.count()/length() to size()

This is semantic patch using ClangTidyTransformator:

  auto QtContainerClass = expr(hasType(namedDecl(hasAnyName(<classes>)))).bind(o)
  makeRule(cxxMemberCallExpr(on(QtContainerClass),
                             callee(cxxMethodDecl(hasAnyName({"count", "length"),
                                                  parameterCountIs(0))))),
           changeTo(cat(access(o, cat("size"), "()"))),
           cat("use 'size()' instead of 'count()/length()'"))

a.k.a qt-port-to-std-compatible-api with config Scope: 'Container'.

<classes> are:

    // sequential:
    "QByteArray",
    "QList",
    "QQueue",
    "QStack",
    "QString",
    "QVarLengthArray",
    "QVector",
    // associative:
    "QHash",
    "QMultiHash",
    "QMap",
    "QMultiMap",
    "QSet",
    // Qt has no QMultiSet

Change-Id: Ibe8837be96e8d30d1846881ecd65180c1bc459af
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
---
 src/sql/kernel/qsqldriver.cpp |  4 ++--
 src/sql/kernel/qsqlrecord.cpp |  6 +++---
 src/sql/kernel/qsqlresult.cpp | 18 +++++++++---------
 3 files changed, 14 insertions(+), 14 deletions(-)

(limited to 'src/sql/kernel')

diff --git a/src/sql/kernel/qsqldriver.cpp b/src/sql/kernel/qsqldriver.cpp
index 997441ab49..7c8c329d64 100644
--- a/src/sql/kernel/qsqldriver.cpp
+++ b/src/sql/kernel/qsqldriver.cpp
@@ -516,7 +516,7 @@ QString QSqlDriver::sqlStatement(StatementType type, const QString &tableName,
             s.clear();
         } else {
             vals.chop(2); // remove trailing comma
-            s[s.length() - 2] = u')';
+            s[s.size() - 2] = u')';
             s.append("VALUES ("_L1).append(vals).append(u')');
         }
         break; }
@@ -598,7 +598,7 @@ QString QSqlDriver::formatValue(const QSqlField &field, bool trimStrings) const
         {
             QString result = field.value().toString();
             if (trimStrings) {
-                int end = result.length();
+                int end = result.size();
                 while (end && result.at(end-1).isSpace()) /* skip white space from end */
                     end--;
                 result.truncate(end);
diff --git a/src/sql/kernel/qsqlrecord.cpp b/src/sql/kernel/qsqlrecord.cpp
index 41fcd745cd..74f42b2e48 100644
--- a/src/sql/kernel/qsqlrecord.cpp
+++ b/src/sql/kernel/qsqlrecord.cpp
@@ -18,7 +18,7 @@ public:
     QSqlRecordPrivate();
     QSqlRecordPrivate(const QSqlRecordPrivate &other);
 
-    inline bool contains(int index) { return index >= 0 && index < fields.count(); }
+    inline bool contains(int index) { return index >= 0 && index < fields.size(); }
     QString createField(int index, const QString &prefix) const;
 
     QList<QSqlField> fields;
@@ -337,7 +337,7 @@ bool QSqlRecord::contains(const QString& name) const
 void QSqlRecord::clearValues()
 {
     detach();
-    int count = d->fields.count();
+    int count = d->fields.size();
     for (int i = 0; i < count; ++i)
         d->fields[i].clear();
 }
@@ -451,7 +451,7 @@ bool QSqlRecord::isGenerated(int index) const
 
 int QSqlRecord::count() const
 {
-    return d->fields.count();
+    return d->fields.size();
 }
 
 /*!
diff --git a/src/sql/kernel/qsqlresult.cpp b/src/sql/kernel/qsqlresult.cpp
index b45f7b6790..56d5edb39f 100644
--- a/src/sql/kernel/qsqlresult.cpp
+++ b/src/sql/kernel/qsqlresult.cpp
@@ -639,13 +639,13 @@ bool QSqlResult::exec()
             else
                 f.setValue(val);
             query = query.replace(d->holders.at(i).holderPos,
-                                   holder.length(), driver()->formatValue(f));
+                                   holder.size(), driver()->formatValue(f));
         }
     } else {
         QString val;
         qsizetype i = 0;
         int idx = 0;
-        for (idx = 0; idx < d->values.count(); ++idx) {
+        for (idx = 0; idx < d->values.size(); ++idx) {
             i = query.indexOf(u'?', i);
             if (i == -1)
                 continue;
@@ -657,7 +657,7 @@ bool QSqlResult::exec()
                 f.setValue(var);
             val = driver()->formatValue(f);
             query = query.replace(i, 1, driver()->formatValue(f));
-            i += val.length();
+            i += val.size();
         }
     }
 
@@ -683,7 +683,7 @@ void QSqlResult::bindValue(int index, const QVariant& val, QSql::ParamType param
     QList<int> &indexes = d->indexes[d->fieldSerial(index)];
     if (!indexes.contains(index))
         indexes.append(index);
-    if (d->values.count() <= index)
+    if (d->values.size() <= index)
         d->values.resize(index + 1);
     d->values[index] = val;
     if (paramType != QSql::In || !d->types.isEmpty())
@@ -709,7 +709,7 @@ void QSqlResult::bindValue(const QString& placeholder, const QVariant& val,
     // bindings - don't reset it
     const QList<int> indexes = d->indexes.value(placeholder);
     for (int idx : indexes) {
-        if (d->values.count() <= idx)
+        if (d->values.size() <= idx)
             d->values.resize(idx + 1);
         d->values[idx] = val;
         if (paramType != QSql::In || !d->types.isEmpty())
@@ -789,7 +789,7 @@ QSql::ParamType QSqlResult::bindValueType(const QString& placeholder) const
 int QSqlResult::boundValueCount() const
 {
     Q_D(const QSqlResult);
-    return d->values.count();
+    return d->values.size();
 }
 
 /*!
@@ -949,10 +949,10 @@ bool QSqlResult::execBatch(bool arrayBind)
     Q_D(QSqlResult);
 
     QList<QVariant> values = d->values;
-    if (values.count() == 0)
+    if (values.size() == 0)
         return false;
-    for (int i = 0; i < values.at(0).toList().count(); ++i) {
-        for (int j = 0; j < values.count(); ++j)
+    for (int i = 0; i < values.at(0).toList().size(); ++i) {
+        for (int j = 0; j < values.size(); ++j)
             bindValue(j, values.at(j).toList().at(i), QSql::In);
         if (!exec())
             return false;
-- 
cgit v1.2.3