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>

1 files changed, 12 insertions, 12 deletions

diff --git a/src/plugins/generic/tuiotouch/qtuiohandler.cpp b/src/plugins/generic/tuiotouch/qtuiohandler.cpp
index 3e67050589..07f0589078 100644
--- a/src/plugins/generic/tuiotouch/qtuiohandler.cpp
+++ b/src/plugins/generic/tuiotouch/qtuiohandler.cpp
@@ -139,7 +139,7 @@ void QTuioHandler::processPackets()
         for (const QOscMessage &message : qAsConst(messages)) {
             if (message.addressPattern() == "/tuio/2Dcur") {
                 QList<QVariant> arguments = message.arguments();
-                if (arguments.count() == 0) {
+                if (arguments.size() == 0) {
                     qCWarning(lcTuioHandler, "Ignoring TUIO message with no arguments");
                     continue;
                 }
@@ -159,7 +159,7 @@ void QTuioHandler::processPackets()
                 }
             } else if (message.addressPattern() == "/tuio/2Dobj") {
                 QList<QVariant> arguments = message.arguments();
-                if (arguments.count() == 0) {
+                if (arguments.size() == 0) {
                     qCWarning(lcTuioHandler, "Ignoring TUIO message with no arguments");
                     continue;
                 }
@@ -188,8 +188,8 @@ void QTuioHandler::processPackets()
 void QTuioHandler::process2DCurSource(const QOscMessage &message)
 {
     QList<QVariant> arguments = message.arguments();
-    if (arguments.count() != 2) {
-        qCWarning(lcTuioSource) << "Ignoring malformed TUIO source message: " << arguments.count();
+    if (arguments.size() != 2) {
+        qCWarning(lcTuioSource) << "Ignoring malformed TUIO source message: " << arguments.size();
         return;
     }
 
@@ -214,7 +214,7 @@ void QTuioHandler::process2DCurAlive(const QOscMessage &message)
     QMap<int, QTuioCursor> oldActiveCursors = m_activeCursors;
     QMap<int, QTuioCursor> newActiveCursors;
 
-    for (int i = 1; i < arguments.count(); ++i) {
+    for (int i = 1; i < arguments.size(); ++i) {
         if (QMetaType::Type(arguments.at(i).userType()) != QMetaType::Int) {
             qCWarning(lcTuioHandler) << "Ignoring malformed TUIO alive message (bad argument on position" << i << arguments << ')';
             return;
@@ -255,8 +255,8 @@ void QTuioHandler::process2DCurAlive(const QOscMessage &message)
 void QTuioHandler::process2DCurSet(const QOscMessage &message)
 {
     QList<QVariant> arguments = message.arguments();
-    if (arguments.count() < 7) {
-        qCWarning(lcTuioSet) << "Ignoring malformed TUIO set message with too few arguments: " << arguments.count();
+    if (arguments.size() < 7) {
+        qCWarning(lcTuioSet) << "Ignoring malformed TUIO set message with too few arguments: " << arguments.size();
         return;
     }
 
@@ -353,8 +353,8 @@ void QTuioHandler::process2DCurFseq(const QOscMessage &message)
 void QTuioHandler::process2DObjSource(const QOscMessage &message)
 {
     QList<QVariant> arguments = message.arguments();
-    if (arguments.count() != 2) {
-        qCWarning(lcTuioSource, ) << "Ignoring malformed TUIO source message: " << arguments.count();
+    if (arguments.size() != 2) {
+        qCWarning(lcTuioSource ) << "Ignoring malformed TUIO source message: " << arguments.size();
         return;
     }
 
@@ -379,7 +379,7 @@ void QTuioHandler::process2DObjAlive(const QOscMessage &message)
     QMap<int, QTuioToken> oldActiveTokens = m_activeTokens;
     QMap<int, QTuioToken> newActiveTokens;
 
-    for (int i = 1; i < arguments.count(); ++i) {
+    for (int i = 1; i < arguments.size(); ++i) {
         if (QMetaType::Type(arguments.at(i).userType()) != QMetaType::Int) {
             qCWarning(lcTuioHandler) << "Ignoring malformed TUIO alive message (bad argument on position" << i << arguments << ')';
             return;
@@ -420,8 +420,8 @@ void QTuioHandler::process2DObjAlive(const QOscMessage &message)
 void QTuioHandler::process2DObjSet(const QOscMessage &message)
 {
     QList<QVariant> arguments = message.arguments();
-    if (arguments.count() < 7) {
-        qCWarning(lcTuioSet) << "Ignoring malformed TUIO set message with too few arguments: " << arguments.count();
+    if (arguments.size() < 7) {
+        qCWarning(lcTuioSet) << "Ignoring malformed TUIO set message with too few arguments: " << arguments.size();
         return;
     }