Code cosmetics

Use 'ids' as name of a list of ids, and the more compact contains()
instead of indexOf() != -1.

Change-Id: Ibfb35ae9d09ac998110d31499918416d4c35f08c
Reviewed-by: Maurice Kalinowski <maurice.kalinowski@qt.io>

2 files changed, 5 insertions, 5 deletions

diff --git a/src/mqtt/qmqttpublishproperties.cpp b/src/mqtt/qmqttpublishproperties.cpp
index e358d61..c04cadd 100644
--- a/src/mqtt/qmqttpublishproperties.cpp
+++ b/src/mqtt/qmqttpublishproperties.cpp
@@ -254,16 +254,16 @@ QList<quint32> QMqttPublishProperties::subscriptionIdentifiers() const
 }
 
 /*!
-    Sets the subscription identifiers to \a id.
+    Sets the subscription identifiers to \a ids.
 */
-void QMqttPublishProperties::setSubscriptionIdentifiers(const QList<quint32> &id)
+void QMqttPublishProperties::setSubscriptionIdentifiers(const QList<quint32> &ids)
 {
-    if (id.indexOf(0) != -1) {
+    if (ids.contains(0)) {
         qCDebug(lcMqttClient) << "A subscription identifier with value 0 is not allowed.";
         return;
     }
     data->details |= QMqttPublishProperties::SubscriptionIdentifier;
-    data->subscriptionIdentifier = id;
+    data->subscriptionIdentifier = ids;
 }
 
 /*!
diff --git a/src/mqtt/qmqttpublishproperties.h b/src/mqtt/qmqttpublishproperties.h
index 93ec6f9..8549fc2 100644
--- a/src/mqtt/qmqttpublishproperties.h
+++ b/src/mqtt/qmqttpublishproperties.h
@@ -83,7 +83,7 @@ public:
     void setUserProperties(const QMqttUserProperties &properties);
 
     QList<quint32> subscriptionIdentifiers() const;
-    void setSubscriptionIdentifiers(const QList<quint32> &id);
+    void setSubscriptionIdentifiers(const QList<quint32> &ids);
 
     QString contentType() const;
     void setContentType(const QString &type);