summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKarsten Heimrich <karsten.heimrich@qt.io>2018-12-11 14:14:50 +0100
committerKarsten Heimrich <karsten.heimrich@qt.io>2018-12-11 15:07:14 +0000
commit3c6e84d7b39014d5df6750f096bc31a5b24e164b (patch)
treea54c04622ff7eef17c2f27536569069dfcd559ed
parent74c70671f8bd1f7138b136048f86eba1e940d8a3 (diff)
Fix some minor warnings spotted by clang
Change-Id: Idd2d3aa9b202957e03917923784c2348ed371c0b Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
-rw-r--r--tests/auto/qknxcontrolfield/tst_qknxcontrolfield.cpp6
-rw-r--r--tests/auto/qknxproject/tst_qknxproject.cpp16
-rw-r--r--tests/auto/qknxtpdufactory/tst_qknxtpdufactory.cpp2
3 files changed, 9 insertions, 15 deletions
diff --git a/tests/auto/qknxcontrolfield/tst_qknxcontrolfield.cpp b/tests/auto/qknxcontrolfield/tst_qknxcontrolfield.cpp
index 594eec9..546fdac 100644
--- a/tests/auto/qknxcontrolfield/tst_qknxcontrolfield.cpp
+++ b/tests/auto/qknxcontrolfield/tst_qknxcontrolfield.cpp
@@ -31,12 +31,6 @@
#include <QtKnx/qknxextendedcontrolfield.h>
#include <QtTest/qtest.h>
-static QString s_msg;
-static void myMessageHandler(QtMsgType, const QMessageLogContext &, const QString &msg)
-{
- s_msg = msg;
-}
-
class tst_QKnxControlField : public QObject
{
Q_OBJECT
diff --git a/tests/auto/qknxproject/tst_qknxproject.cpp b/tests/auto/qknxproject/tst_qknxproject.cpp
index a9429dd..9e69d61 100644
--- a/tests/auto/qknxproject/tst_qknxproject.cpp
+++ b/tests/auto/qknxproject/tst_qknxproject.cpp
@@ -76,7 +76,7 @@ private Q_SLOTS:
QCOMPARE(projectInfo.CodePage, QLatin1String("us-ascii"));
QCOMPARE(projectInfo.BusAccessLegacyMode, true);
QCOMPARE(projectInfo.Guid, QLatin1String("eae8eae4-34f3-433b-bb22-5acdba366889"));
- QCOMPARE(projectInfo.LastUsedPuid, 47);
+ QCOMPARE(projectInfo.LastUsedPuid, 47u);
QCOMPARE(projectInfo.HistoryEntries.size(), 2);
QCOMPARE(projectInfo.HistoryEntries[0].Date,
@@ -316,7 +316,7 @@ private Q_SLOTS:
auto address = subRange.GroupAddress[0];
QCOMPARE(address.Id, QLatin1String("P-050F-0_GA-355"));
- QCOMPARE(address.Address, 1);
+ QCOMPARE(address.Address, 1u);
QCOMPARE(address.Name, QLatin1String("Name"));
QCOMPARE(address.Description, QLatin1String("Description"));
QCOMPARE(address.DatapointType, QLatin1String("DPST-1-1"));
@@ -331,7 +331,7 @@ private Q_SLOTS:
address = subRange.GroupAddress[1];
QCOMPARE(address.Id, QLatin1String("P-050F-0_GA-356"));
- QCOMPARE(address.Address, 2);
+ QCOMPARE(address.Address, 2u);
QCOMPARE(address.Name, QLatin1String("Name2"));
QCOMPARE(address.Description, QLatin1String("Description2"));
QCOMPARE(address.DatapointType, QLatin1String("DPST-20-102"));
@@ -361,7 +361,7 @@ private Q_SLOTS:
address = groupRange.GroupAddress[0];
QCOMPARE(address.Id, QLatin1String("P-050F-0_GA-261"));
- QCOMPARE(address.Address, 2048);
+ QCOMPARE(address.Address, 2048u);
QCOMPARE(address.Name, QLatin1String("Name3"));
QCOMPARE(address.Description, QLatin1String("Description3"));
QCOMPARE(address.DatapointType, QLatin1String("DPST-1-1"));
@@ -391,7 +391,7 @@ private Q_SLOTS:
address = subRange.GroupAddress[0];
QCOMPARE(address.Id, QLatin1String("P-050F-0_GA-262"));
- QCOMPARE(address.Address, 2049);
+ QCOMPARE(address.Address, 2049u);
QCOMPARE(address.Name, QLatin1String("Name4"));
QCOMPARE(address.Description, QLatin1String("Description4"));
QCOMPARE(address.DatapointType, QLatin1String("DPST-1-1"));
@@ -421,7 +421,7 @@ private Q_SLOTS:
address = subRange.GroupAddress[0];
QCOMPARE(address.Id, QLatin1String("P-050F-0_GA-358"));
- QCOMPARE(address.Address, 2304);
+ QCOMPARE(address.Address, 2304u);
QCOMPARE(address.Name, QLatin1String("Name5"));
QCOMPARE(address.Description, QLatin1String("Description5"));
QCOMPARE(address.DatapointType, QLatin1String("DPST-1-1"));
@@ -436,7 +436,7 @@ private Q_SLOTS:
address = subRange.GroupAddress[1];
QCOMPARE(address.Id, QLatin1String("P-050F-0_GA-359"));
- QCOMPARE(address.Address, 2305);
+ QCOMPARE(address.Address, 2305u);
QCOMPARE(address.Name, QLatin1String("Name6"));
QCOMPARE(address.Description, QLatin1String("Description6"));
QCOMPARE(address.DatapointType, QString());
@@ -765,7 +765,7 @@ private Q_SLOTS:
QCOMPARE(devInstance.Security.LoadedDeviceManagementPassword, QLatin1String("LoadedMngmtPassword9"));
QCOMPARE(devInstance.Security.LoadedIPRoutingBackboneKey, QLatin1String("LoadedIPRoutingBackboneKey9"));
QCOMPARE(devInstance.Security.LoadedToolKey, QLatin1String("LoadedToolKey9"));
- QCOMPARE(devInstance.Security.SequenceNumber, 9);
+ QCOMPARE(devInstance.Security.SequenceNumber, 9u);
QCOMPARE(devInstance.Security.SequenceNumberTimestamp,
QDateTime::fromString(QLatin1String("2009-09-09T15:02:26.2503409Z"), Qt::ISODateWithMs));
QCOMPARE(devInstance.Security.ToolKey, QLatin1String("ToolKey9"));
diff --git a/tests/auto/qknxtpdufactory/tst_qknxtpdufactory.cpp b/tests/auto/qknxtpdufactory/tst_qknxtpdufactory.cpp
index dcd6e6b..454f61f 100644
--- a/tests/auto/qknxtpdufactory/tst_qknxtpdufactory.cpp
+++ b/tests/auto/qknxtpdufactory/tst_qknxtpdufactory.cpp
@@ -219,7 +219,7 @@ void tst_QKnxTpduFactory::testTpdu()
QCOMPARE(tpdu.bytes(), QKnxByteArray({ 0x00, 0xbf }));
tpdu.setData({ 0x43 });
- QCOMPARE(tpdu.data(), QKnxByteArray(1, { 0x43 }));
+ QCOMPARE(tpdu.data(), QKnxByteArray(1, 0x43));
QCOMPARE(tpdu.bytes(), QKnxByteArray({ 0x00, 0x80, 0x43 }));
tpdu = { QKnxTpdu::TransportControlField::DataGroup,