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>

2 files changed, 15 insertions, 15 deletions

diff --git a/src/corelib/tools/qcommandlineparser.cpp b/src/corelib/tools/qcommandlineparser.cpp
index e771877265..9f890fa8be 100644
--- a/src/corelib/tools/qcommandlineparser.cpp
+++ b/src/corelib/tools/qcommandlineparser.cpp
@@ -691,7 +691,7 @@ bool QCommandLineParserPrivate::parse(const QStringList &args)
         if (forcePositional) {
             positionalArgumentList.append(argument);
         } else if (argument.startsWith(doubleDashString)) {
-            if (argument.length() > 2) {
+            if (argument.size() > 2) {
                 QString optionName = argument.mid(2).section(assignChar, 0, 0);
                 if (registerFoundOption(optionName)) {
                     if (!parseOptionValue(optionName, argument, &argumentIterator, args.end()))
@@ -1042,7 +1042,7 @@ static QString wrapText(const QString &names, int optionNameMaxWidth, const QStr
     int lastBreakable = -1;
     const int max = 79 - (indentation.size() + optionNameMaxWidth + 1);
     int x = 0;
-    const int len = description.length();
+    const int len = description.size();
 
     for (int i = 0; i < len; ++i) {
         ++x;
@@ -1116,7 +1116,7 @@ QString QCommandLineParserPrivate::helpText(bool includeQtOptions) const
         const QStringList optionNames = option.names();
         QString optionNamesString;
         for (const QString &optionName : optionNames) {
-            const int numDashes = optionName.length() == 1 ? 1 : 2;
+            const int numDashes = optionName.size() == 1 ? 1 : 2;
             optionNamesString += QLatin1StringView("--", numDashes) + optionName + ", "_L1;
         }
         if (!optionNames.isEmpty())
@@ -1125,7 +1125,7 @@ QString QCommandLineParserPrivate::helpText(bool includeQtOptions) const
         if (!valueName.isEmpty())
             optionNamesString += " <"_L1 + valueName + u'>';
         optionNameList.append(optionNamesString);
-        longestOptionNameString = qMax(longestOptionNameString, optionNamesString.length());
+        longestOptionNameString = qMax(longestOptionNameString, optionNamesString.size());
     }
     ++longestOptionNameString;
     const int optionNameMaxWidth = qMin(50, longestOptionNameString);
diff --git a/src/corelib/tools/qeasingcurve.cpp b/src/corelib/tools/qeasingcurve.cpp
index 9242a617ba..40041a9ff3 100644
--- a/src/corelib/tools/qeasingcurve.cpp
+++ b/src/corelib/tools/qeasingcurve.cpp
@@ -447,7 +447,7 @@ struct BezierEase : public QEasingCurveFunction
     {
         if (_bezierCurves.constLast() == QPointF(1.0, 1.0)) {
             _init = true;
-            _curveCount = _bezierCurves.count() / 3;
+            _curveCount = _bezierCurves.size() / 3;
 
             for (int i=0; i < _curveCount; i++) {
                 _intervals[i] = _bezierCurves.at(i * 3 + 2).x();
@@ -466,17 +466,17 @@ struct BezierEase : public QEasingCurveFunction
                     _curves[0].p3y = _bezierCurves.at(2).y();
 
                 } else if (i == (_curveCount - 1)) {
-                    _curves[i].p0x = _bezierCurves.at(_bezierCurves.count() - 4).x();
-                    _curves[i].p0y = _bezierCurves.at(_bezierCurves.count() - 4).y();
+                    _curves[i].p0x = _bezierCurves.at(_bezierCurves.size() - 4).x();
+                    _curves[i].p0y = _bezierCurves.at(_bezierCurves.size() - 4).y();
 
-                    _curves[i].p1x = _bezierCurves.at(_bezierCurves.count() - 3).x();
-                    _curves[i].p1y = _bezierCurves.at(_bezierCurves.count() - 3).y();
+                    _curves[i].p1x = _bezierCurves.at(_bezierCurves.size() - 3).x();
+                    _curves[i].p1y = _bezierCurves.at(_bezierCurves.size() - 3).y();
 
-                    _curves[i].p2x = _bezierCurves.at(_bezierCurves.count() - 2).x();
-                    _curves[i].p2y = _bezierCurves.at(_bezierCurves.count() - 2).y();
+                    _curves[i].p2x = _bezierCurves.at(_bezierCurves.size() - 2).x();
+                    _curves[i].p2y = _bezierCurves.at(_bezierCurves.size() - 2).y();
 
-                    _curves[i].p3x = _bezierCurves.at(_bezierCurves.count() - 1).x();
-                    _curves[i].p3y = _bezierCurves.at(_bezierCurves.count() - 1).y();
+                    _curves[i].p3x = _bezierCurves.at(_bezierCurves.size() - 1).x();
+                    _curves[i].p3y = _bezierCurves.at(_bezierCurves.size() - 1).y();
                 } else {
                     _curves[i].p0x = _bezierCurves.at(i * 3 - 1).x();
                     _curves[i].p0y = _bezierCurves.at(i * 3 - 1).y();
@@ -535,7 +535,7 @@ struct BezierEase : public QEasingCurveFunction
 
     qreal value(qreal x) override
     {
-        Q_ASSERT(_bezierCurves.count() % 3 == 0);
+        Q_ASSERT(_bezierCurves.size() % 3 == 0);
 
         if (_bezierCurves.isEmpty()) {
             return x;
@@ -869,7 +869,7 @@ struct TCBEase : public BezierEase
 
     qreal value(qreal x) override
     {
-        Q_ASSERT(_bezierCurves.count() % 3 == 0);
+        Q_ASSERT(_bezierCurves.size() % 3 == 0);
 
         if (_bezierCurves.isEmpty()) {
             qWarning("QEasingCurve: Invalid tcb curve");