Clean-up warnings in corelib/thread tests

Fixed the following warnings:

 - empty expression statement has no effect; remove unnecessary ';'
   to silence this warning
 - zero as null pointer constant
 - no previous extern declaration for non-static variable
 - use of old-style cast
 - integer conversion warnings

Change-Id: Ieb31b127ce8e3b543aaf88a8e8fc463dcc8bc4e3
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>

1 files changed, 2 insertions, 2 deletions

diff --git a/tests/auto/corelib/thread/qpromise/tst_qpromise.cpp b/tests/auto/corelib/thread/qpromise/tst_qpromise.cpp
index 82a359451d..74a3059699 100644
--- a/tests/auto/corelib/thread/qpromise/tst_qpromise.cpp
+++ b/tests/auto/corelib/thread/qpromise/tst_qpromise.cpp
@@ -77,7 +77,7 @@ private slots:
 
 struct TrivialType { int field = 0; };
 struct CopyOnlyType {
-    Q_DISABLE_MOVE(CopyOnlyType);
+    Q_DISABLE_MOVE(CopyOnlyType)
     CopyOnlyType() = default;
     CopyOnlyType(const CopyOnlyType &) = default;
     CopyOnlyType& operator=(const CopyOnlyType &) = default;
@@ -86,7 +86,7 @@ struct CopyOnlyType {
     int field = 0;
 };
 struct MoveOnlyType {
-    Q_DISABLE_COPY(MoveOnlyType);
+    Q_DISABLE_COPY(MoveOnlyType)
     MoveOnlyType() = default;
     MoveOnlyType(MoveOnlyType &&) = default;
     MoveOnlyType& operator=(MoveOnlyType &&) = default;